n8n/packages/workflow/test/expression-sandboxing.test.ts
n8n-assistant[bot] 562d867483
chore: Bundle 2026-W7 (#26214)
Signed-off-by: Oleg Ivaniv <me@olegivaniv.com>
Co-authored-by: Tomi Turtiainen <10324676+tomi@users.noreply.github.com>
Co-authored-by: yehorkardash <yehor.kardash@n8n.io>
Co-authored-by: James Gee <1285296+geemanjs@users.noreply.github.com>
Co-authored-by: Iván Ovejero <ivov.src@gmail.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Stephen Wright <sjw948@gmail.com>
Co-authored-by: oleg <me@olegivaniv.com>
Co-authored-by: Albert Alises <albert.alises@gmail.com>
Co-authored-by: Danny Martini <danny@n8n.io>
2026-02-25 07:08:58 +00:00

995 lines
30 KiB
TypeScript

import { Tournament } from '@n8n/tournament';
import {
DollarSignValidator,
ThisSanitizer,
PrototypeSanitizer,
sanitizer,
DOLLAR_SIGN_ERROR,
} from '../src/expression-sandboxing';
import {
ExpressionClassExtensionError,
ExpressionComputedDestructuringError,
ExpressionDestructuringError,
ExpressionError,
ExpressionWithStatementError,
} from '../src/errors';
const tournament = new Tournament(
(e) => {
throw e;
},
undefined,
undefined,
{
before: [ThisSanitizer],
after: [PrototypeSanitizer, DollarSignValidator],
},
);
const errorRegex = /^Cannot access ".*" due to security concerns$/;
describe('PrototypeSanitizer', () => {
describe('Static analysis', () => {
it('should not allow access to __proto__', () => {
expect(() => {
tournament.execute('{{ ({}).__proto__.__proto__ }}', {});
}).toThrowError(errorRegex);
expect(() => {
tournament.execute('{{ ({})["__proto__"]["__proto__"] }}', {});
}).toThrowError(errorRegex);
});
it('should not allow access to prototype', () => {
expect(() => {
tournament.execute('{{ Number.prototype }}', { Number });
}).toThrowError(errorRegex);
expect(() => {
tournament.execute('{{ Number["prototype"] }}', { Number });
}).toThrowError(errorRegex);
});
it('should not allow access to constructor', () => {
expect(() => {
tournament.execute('{{ Number.constructor }}', {
__sanitize: sanitizer,
Number,
});
}).toThrowError(errorRegex);
expect(() => {
tournament.execute('{{ Number["constructor"] }}', {
__sanitize: sanitizer,
Number,
});
}).toThrowError(errorRegex);
});
it.each([
['dot notation', '{{ Error.prepareStackTrace }}'],
['bracket notation', '{{ Error["prepareStackTrace"] }}'],
['assignment', '{{ Error.prepareStackTrace = (e, s) => s }}'],
])('should not allow access to prepareStackTrace via %s', (_, expression) => {
expect(() => {
tournament.execute(expression, { __sanitize: sanitizer, Error });
}).toThrowError(errorRegex);
});
it.each([
['constructor', '{{ Number[`constructor`] }}', { Number }],
// eslint-disable-next-line n8n-local-rules/no-interpolation-in-regular-string
['constructor (Number)', '{{ Number[`constr${`uct`}or`] }}', { Number }],
// eslint-disable-next-line n8n-local-rules/no-interpolation-in-regular-string
['constructor (Object)', "{{ Object[`constr${'uct'}or`] }}", { Object }],
['__proto__', '{{ ({})[`__proto__`] }}', {}],
['mainModule', '{{ process[`mainModule`] }}', { process: {} }],
])('should not allow access to %s via template literal', (_, expression, context) => {
expect(() => {
tournament.execute(expression, { __sanitize: sanitizer, ...context });
}).toThrowError(errorRegex);
});
it.each([
['getPrototypeOf', '{{ Object.getPrototypeOf }}'],
['binding', '{{ process.binding }}'],
['_load', '{{ module._load }}'],
])('should not allow access to %s', (_, expression) => {
expect(() => {
tournament.execute(expression, { __sanitize: sanitizer, Object, process: {}, module: {} });
}).toThrowError(errorRegex);
});
it.each([
['dot notation', '{{ (()=>{}).caller }}'],
['bracket notation', '{{ (()=>{})["caller"] }}'],
])('should not allow access to caller via %s', (_, expression) => {
expect(() => {
tournament.execute(expression, { __sanitize: sanitizer });
}).toThrowError(errorRegex);
});
it.each([
['dot notation', '{{ (()=>{}).arguments }}'],
['bracket notation', '{{ (()=>{})["arguments"] }}'],
])('should not allow access to arguments via %s', (_, expression) => {
expect(() => {
tournament.execute(expression, { __sanitize: sanitizer });
}).toThrowError(errorRegex);
});
it.each([
['getBuiltinModule', '{{ ({}).getBuiltinModule }}'],
['_linkedBinding', '{{ ({})._linkedBinding }}'],
['dlopen', '{{ ({}).dlopen }}'],
['execve', '{{ ({}).execve }}'],
['loadEnvFile', '{{ ({}).loadEnvFile }}'],
])('should not allow access to %s', (_, expression) => {
expect(() => {
tournament.execute(expression, { __sanitize: sanitizer });
}).toThrowError(errorRegex);
});
describe('Dollar sign identifier handling', () => {
it('should not allow bare $ identifier', () => {
expect(() => {
tournament.execute('{{ $ }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
expect(() => {
tournament.execute('{{$}}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
});
it('should not allow $ in expressions', () => {
expect(() => {
tournament.execute('{{ "prefix" + $ }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
expect(() => {
tournament.execute('{{ $ + "suffix" }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
expect(() => {
tournament.execute('{{ 1 + $ }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
expect(() => {
tournament.execute('{{ [1, 2, $] }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
expect(() => {
tournament.execute('{{ {value: $} }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
});
it('should not allow $ with property access', () => {
expect(() => {
tournament.execute('{{ $.something }}', { $: { something: 'value' } });
}).toThrowError(DOLLAR_SIGN_ERROR);
expect(() => {
tournament.execute('{{ $["property"] }}', { $: { property: 'value' } });
}).toThrowError(DOLLAR_SIGN_ERROR);
});
it('should allow $ as function call', () => {
const mockFunction = () => 'result';
expect(() => {
tournament.execute('{{ $() }}', { $: mockFunction });
}).not.toThrow();
expect(() => {
tournament.execute('{{ $("node_name") }}', { $: mockFunction });
}).not.toThrow();
expect(() => {
tournament.execute('{{ $().someMethod() }}', { $: () => ({ someMethod: () => 'test' }) });
}).not.toThrow();
});
it('should allow $ in strings', () => {
expect(() => {
tournament.execute('{{ "test$test" }}', {});
}).not.toThrow();
expect(() => {
tournament.execute("{{ 'price: $100' }}", {});
}).not.toThrow();
expect(() => {
// eslint-disable-next-line n8n-local-rules/no-interpolation-in-regular-string
tournament.execute('{{ `template ${100}$` }}', {});
}).not.toThrow();
});
it('should allow $ as part of variable names', () => {
expect(() => {
tournament.execute('{{ $json }}', { $json: { test: 'value' } });
}).not.toThrow();
expect(() => {
tournament.execute('{{ price$ }}', { price$: 100 });
}).not.toThrow();
expect(() => {
tournament.execute('{{ my$var }}', { my$var: 'test' });
}).not.toThrow();
expect(() => {
tournament.execute('{{ _$_ }}', { _$_: 'underscore' });
}).not.toThrow();
});
it('should allow $ as a property name', () => {
// $ is a valid property name in JavaScript, so obj.$ should be allowed
expect(() => {
tournament.execute('{{ obj.$ }}', { obj: { $: 'value' } });
}).not.toThrow();
expect(() => {
tournament.execute('{{ data["$"] }}', { data: { $: 'value' } });
}).not.toThrow();
expect(() => {
const obj = { nested: { $: 'deep' } };
tournament.execute('{{ obj.nested.$ }}', { obj });
}).not.toThrow();
});
it('should allow $ in conditional expressions with function calls', () => {
const mockFunction = () => 'result';
expect(() => {
tournament.execute('{{ true ? $() : "fallback" }}', { $: mockFunction });
}).not.toThrow();
expect(() => {
tournament.execute('{{ $() || "default" }}', { $: mockFunction });
}).not.toThrow();
expect(() => {
tournament.execute('{{ $() && "continue" }}', { $: mockFunction });
}).not.toThrow();
});
it('should not allow $ in conditional expressions without function calls', () => {
expect(() => {
tournament.execute('{{ true ? $ : "fallback" }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
expect(() => {
tournament.execute('{{ $ || "default" }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
expect(() => {
tournament.execute('{{ $ && "continue" }}', { $: () => 'test' });
}).toThrowError(DOLLAR_SIGN_ERROR);
});
});
});
describe('Runtime', () => {
it('should not allow access to __proto__', () => {
expect(() => {
tournament.execute('{{ ({})["__" + (() => "proto")() + "__"] }}', {
__sanitize: sanitizer,
});
}).toThrowError(errorRegex);
});
it('should not allow access to prototype', () => {
expect(() => {
tournament.execute('{{ Number["pro" + (() => "toty")() + "pe"] }}', {
__sanitize: sanitizer,
Number,
});
}).toThrowError(errorRegex);
});
it('should not allow access to constructor', () => {
expect(() => {
tournament.execute('{{ Number["cons" + (() => "truc")() + "tor"] }}', {
__sanitize: sanitizer,
Number,
});
}).toThrowError(errorRegex);
});
it('should not allow access to caller via concatenation', () => {
expect(() => {
tournament.execute('{{ (()=>{})["cal" + "ler"] }}', {
__sanitize: sanitizer,
});
}).toThrowError(errorRegex);
});
it('should not allow access to arguments via concatenation', () => {
expect(() => {
tournament.execute('{{ (()=>{})["arg" + "uments"] }}', {
__sanitize: sanitizer,
});
}).toThrowError(errorRegex);
});
describe('Array-based property access bypass attempts', () => {
it('should not allow access to __proto__ via array', () => {
expect(() => {
tournament.execute('{{ ({})[["__proto__"]] }}', {
__sanitize: sanitizer,
});
}).toThrowError(errorRegex);
});
it('should not allow access to constructor via array', () => {
expect(() => {
tournament.execute('{{ ({})[["constructor"]] }}', {
__sanitize: sanitizer,
});
}).toThrowError(errorRegex);
});
it('should not allow access to prototype via array', () => {
expect(() => {
tournament.execute('{{ Number[["prototype"]] }}', {
__sanitize: sanitizer,
Number,
});
}).toThrowError(errorRegex);
});
it('should not allow prototype pollution via array access', () => {
expect(() => {
tournament.execute('{{ ({})[["__proto__"]].polluted = 1 }}', {
__sanitize: sanitizer,
});
}).toThrowError(errorRegex);
});
it('should not allow RCE via chained array access', () => {
expect(() => {
tournament.execute('{{ ({})[["toString"]][["constructor"]]("return 1")() }}', {
__sanitize: sanitizer,
});
}).toThrowError(errorRegex);
});
it('should not allow access to prepareStackTrace via array', () => {
expect(() => {
tournament.execute('{{ Error[["prepareStackTrace"]] }}', {
__sanitize: sanitizer,
Error,
});
}).toThrowError(errorRegex);
});
});
});
describe('Class extension bypass attempts', () => {
it('should not allow class extending Function', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Z extends Function {} return new Z("return 1")(); })() }}',
{ __sanitize: sanitizer },
);
}).toThrowError(ExpressionClassExtensionError);
});
it('should not allow class expression extending Function', () => {
expect(() => {
tournament.execute(
'{{ (() => { const Z = class extends Function {}; return new Z("return 1")(); })() }}',
{ __sanitize: sanitizer },
);
}).toThrowError(ExpressionClassExtensionError);
});
it('should not allow class extending GeneratorFunction', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Z extends GeneratorFunction {} return new Z("yield 1"); })() }}',
{ __sanitize: sanitizer },
);
}).toThrowError(ExpressionClassExtensionError);
});
it('should not allow class extending AsyncFunction', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Z extends AsyncFunction {} return new Z("return 1"); })() }}',
{ __sanitize: sanitizer },
);
}).toThrowError(ExpressionClassExtensionError);
});
it('should not allow class extending AsyncGeneratorFunction', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Z extends AsyncGeneratorFunction {} return new Z("yield 1"); })() }}',
{ __sanitize: sanitizer },
);
}).toThrowError(ExpressionClassExtensionError);
});
it('should allow class extending safe classes', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Child extends Array {} return new Child(1, 2, 3).length; })() }}',
{ __sanitize: sanitizer, Array },
);
}).not.toThrow();
});
it('should allow class without extends', () => {
expect(() => {
tournament.execute('{{ (() => { class MyClass {} return new MyClass(); })() }}', {
__sanitize: sanitizer,
});
}).not.toThrow();
});
it('should not allow class extending via CallExpression bypass', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Z extends (() => Function)() {} return new Z("return 1")(); })() }}',
{ __sanitize: sanitizer, Function },
);
}).toThrowError(ExpressionError);
});
it('should not allow class expression extending via CallExpression bypass', () => {
expect(() => {
tournament.execute(
'{{ (() => { const Z = class extends (() => Function)() {}; return new Z("return 1")(); })() }}',
{ __sanitize: sanitizer, Function },
);
}).toThrowError(ExpressionError);
});
it('should not allow class extending via ConditionalExpression bypass', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Z extends (true ? Function : Object) {} return new Z("return 1")(); })() }}',
{ __sanitize: sanitizer, Function, Object },
);
}).toThrowError(ExpressionError);
});
it('should not allow class extending via SequenceExpression bypass', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Z extends (0, Function) {} return new Z("return 1")(); })() }}',
{ __sanitize: sanitizer, Function },
);
}).toThrowError(ExpressionError);
});
it('should not allow class extending via LogicalExpression bypass', () => {
expect(() => {
tournament.execute(
'{{ (() => { class Z extends (Function || Object) {} return new Z("return 1")(); })() }}',
{ __sanitize: sanitizer, Function, Object },
);
}).toThrowError(ExpressionError);
});
});
describe('Destructuring patterns', () => {
it('should not allow destructuring constructor from arrow function', () => {
expect(() => {
tournament.execute(
'{{ (() => { const {constructor} = ()=>{}; return constructor; })() }}',
{
__sanitize: sanitizer,
},
);
}).toThrowError(ExpressionDestructuringError);
});
it('should not allow destructuring constructor from regular function', () => {
expect(() => {
tournament.execute(
'{{ (() => { const {constructor} = function(){}; return constructor; })() }}',
{
__sanitize: sanitizer,
},
);
}).toThrowError(ExpressionDestructuringError);
});
it('should not allow destructuring constructor with alias', () => {
expect(() => {
tournament.execute('{{ (() => { const {constructor: c} = ()=>{}; return c; })() }}', {
__sanitize: sanitizer,
});
}).toThrowError(ExpressionDestructuringError);
});
it('should not allow destructuring __proto__', () => {
expect(() => {
tournament.execute('{{ (() => { const {__proto__} = {}; return __proto__; })() }}', {
__sanitize: sanitizer,
});
}).toThrowError(ExpressionDestructuringError);
});
it('should not allow destructuring prototype', () => {
expect(() => {
tournament.execute(
'{{ (() => { const {prototype} = function(){}; return prototype; })() }}',
{
__sanitize: sanitizer,
},
);
}).toThrowError(ExpressionDestructuringError);
});
it('should not allow destructuring mainModule', () => {
expect(() => {
tournament.execute('{{ (() => { const {mainModule} = process; return mainModule; })() }}', {
__sanitize: sanitizer,
process: { mainModule: {} },
});
}).toThrowError(ExpressionDestructuringError);
});
it('should not allow destructuring caller', () => {
expect(() => {
tournament.execute('{{ (() => { const {caller} = ()=>{}; return caller; })() }}', {
__sanitize: sanitizer,
});
}).toThrowError(ExpressionDestructuringError);
});
it('should not allow destructuring arguments', () => {
expect(() => {
tournament.execute('{{ (() => { const {arguments: a} = function(){}; return a; })() }}', {
__sanitize: sanitizer,
});
}).toThrowError(ExpressionDestructuringError);
});
it('should allow destructuring safe properties', () => {
const result = tournament.execute(
'{{ (() => { const {name, value} = {name: "test", value: 42}; return name + value; })() }}',
{
__sanitize: sanitizer,
},
);
expect(result).toBe('test42');
});
it('should allow destructuring multiple safe properties', () => {
const result = tournament.execute(
'{{ (() => { const {a, b, c} = {a: 1, b: 2, c: 3}; return a + b + c; })() }}',
{
__sanitize: sanitizer,
},
);
expect(result).toBe(6);
});
it('should not allow computed property destructuring', () => {
expect(() => {
tournament.execute(
'{{ (() => { const a = "constructor"; const {[a]: c} = {}; return c; })() }}',
{
__sanitize: sanitizer,
},
);
}).toThrowError(ExpressionComputedDestructuringError);
});
});
describe('Spread-based global access', () => {
it('should not allow spreading process', () => {
expect(() => {
tournament.execute('{{ ((g) => g.getBuiltinModule)(({...process})) }}', {
__sanitize: sanitizer,
});
}).toThrowError(/due to security concerns/);
});
it('should not allow spreading process in object literal', () => {
expect(() => {
tournament.execute('{{ ({...process}) }}', { __sanitize: sanitizer });
}).toThrowError(/Cannot spread "process" due to security concerns/);
});
it('should not allow spreading process in array', () => {
expect(() => {
tournament.execute('{{ [...process] }}', { __sanitize: sanitizer });
}).toThrowError(/Cannot spread "process" due to security concerns/);
});
it('should not allow spreading global', () => {
expect(() => {
tournament.execute('{{ ({...global}) }}', { __sanitize: sanitizer });
}).toThrowError(/Cannot spread "global" due to security concerns/);
});
it('should not allow spreading Buffer', () => {
expect(() => {
tournament.execute('{{ ({...Buffer}) }}', { __sanitize: sanitizer });
}).toThrowError(/Cannot spread "Buffer" due to security concerns/);
});
it('should not allow the exact RCE PoC payload', () => {
expect(() => {
tournament.execute(
"{{ ((g) => g.getBuiltinModule('child_process').execSync('id').toString())({...process}) }}",
{ __sanitize: sanitizer },
);
}).toThrowError(/due to security concerns/);
});
it('should not allow spreading process in function call arguments', () => {
expect(() => {
tournament.execute('{{ ((a, b) => a)(...process) }}', { __sanitize: sanitizer });
}).toThrowError(/Cannot spread "process" due to security concerns/);
});
it('should not allow spreading process inside arrow function', () => {
expect(() => {
tournament.execute('{{ (() => ({...process}))() }}', { __sanitize: sanitizer });
}).toThrowError(/Cannot spread "process" due to security concerns/);
});
it('should not allow spreading process in nested spread', () => {
expect(() => {
tournament.execute('{{ ({...({...process})}) }}', { __sanitize: sanitizer });
}).toThrowError(/Cannot spread "process" due to security concerns/);
});
it('should not allow spreading process in template expression', () => {
expect(() => {
// eslint-disable-next-line n8n-local-rules/no-interpolation-in-regular-string
tournament.execute('{{ `${JSON.stringify({...process})}` }}', {
__sanitize: sanitizer,
});
}).toThrow();
});
it('should not allow spreading process among other spreads', () => {
expect(() => {
tournament.execute('{{ ({...{a:1}, ...process}) }}', { __sanitize: sanitizer });
}).toThrowError(/Cannot spread "process" due to security concerns/);
});
it('should resolve spread from data context process, not the real one', () => {
const result = tournament.execute('{{ ({...process}).safe }}', {
__sanitize: sanitizer,
process: { safe: true },
});
expect(result).toBe(true);
});
it('should not expose real process.version via spread', () => {
const result = tournament.execute('{{ typeof ({...process}).version }}', {
__sanitize: sanitizer,
process: {},
});
expect(result).toBe('undefined');
});
it('should use data context pid via spread, not real pid', () => {
const result = tournament.execute('{{ ({...process}).pid }}', {
__sanitize: sanitizer,
process: { pid: -1 },
});
expect(result).toBe(-1);
});
it('should use data context when spread is wrapped in arrow function', () => {
const result = tournament.execute('{{ ((g) => g.pid)({...process}) }}', {
__sanitize: sanitizer,
process: { pid: -1 },
});
expect(result).toBe(-1);
});
it('should not give access to real process.exit via spread', () => {
const result = tournament.execute('{{ typeof ({...process}).exit }}', {
__sanitize: sanitizer,
process: {},
});
expect(result).not.toBe('function');
});
it('should not give access to real process.env via spread', () => {
const result = tournament.execute('{{ typeof ({...process}).env }}', {
__sanitize: sanitizer,
process: {},
});
expect(result).not.toBe('object');
});
it('should not give access to getBuiltinModule via spread', () => {
let result: unknown;
try {
result = tournament.execute('{{ typeof ({...process}).getBuiltinModule }}', {
__sanitize: sanitizer,
process: {},
});
} catch {
// Blocked by PrototypeSanitizer — also a valid outcome
return;
}
expect(result).not.toBe('function');
});
});
describe('`with` statement', () => {
it('should not allow `with` statements', () => {
expect(() => {
tournament.execute('{{ (() => { with({}) { return 1; } })() }}', { __sanitize: sanitizer });
}).toThrowError(ExpressionWithStatementError);
});
it('should not allow constructor access via `with` statement', () => {
expect(() => {
tournament.execute(
'{{ (function(){ var constructor = 123; with(function(){}){ return constructor("return 1")() } })() }}',
{ __sanitize: sanitizer },
);
}).toThrowError(ExpressionWithStatementError);
});
it('should not allow RCE via with statement', () => {
expect(() => {
tournament.execute(
"{{ (function(){ var constructor = 123; with(function(){}){ return constructor(\"return process.mainModule.require('child_process').execSync('env').toString().trim()\")() } })() }}",
{
__sanitize: sanitizer,
},
);
}).toThrowError(ExpressionWithStatementError);
});
it('should not allow nested `with` statements', () => {
expect(() => {
tournament.execute('{{ (() => { with({a:1}) { with({b:2}) { return a + b; } } })() }}', {
__sanitize: sanitizer,
});
}).toThrowError(ExpressionWithStatementError);
});
it('should not allow `with` statement accessing prototype chain', () => {
expect(() => {
tournament.execute('{{ (() => { with(Object) { return getPrototypeOf({}); } })() }}', {
__sanitize: sanitizer,
Object,
});
}).toThrowError(ExpressionWithStatementError);
});
});
});
describe('ThisSanitizer', () => {
describe('call expression where callee is function expression', () => {
it('should transform call expression', () => {
const result = tournament.execute('{{ (function() { return this.process; })() }}', {
__sanitize: sanitizer,
});
expect(result).toEqual({});
});
it('should handle recursive call expression', () => {
const result = tournament.execute(
'{{ (function factorial(n) { return n <= 1 ? 1 : n * factorial(n - 1); })(5) }}',
{
__sanitize: sanitizer,
},
);
expect(result).toBe(120);
});
it('should not expose process.env through named function', () => {
const result = tournament.execute('{{ (function test(){ return this.process.env })() }}', {
__sanitize: sanitizer,
});
expect(result).toBe(undefined);
});
it('should still allow access to workflow data via variables', () => {
const result = tournament.execute('{{ (function() { return $json.value; })() }}', {
__sanitize: sanitizer,
$json: { value: 'test-value' },
});
expect(result).toBe('test-value');
});
it('should handle nested call expression', () => {
const result = tournament.execute(
'{{ (function() { return (function() { return this.process; })(); })() }}',
{
__sanitize: sanitizer,
},
);
expect(result).toEqual({});
});
it('should handle nested recursive call expression', () => {
const result = tournament.execute(
'{{ (function() { return (function factorial(n) { return n <= 1 ? 1 : n * factorial(n - 1); })(5); })() }}',
{
__sanitize: sanitizer,
},
);
expect(result).toBe(120);
});
});
describe('function expression', () => {
it('should transform function expression', () => {
const result = tournament.execute('{{ [1].map(function() { return this.process; }) }}', {
__sanitize: sanitizer,
});
expect(result).toEqual([{}]);
});
it('should handle recursive function expression', () => {
const result = tournament.execute(
'{{ [1, 2, 3, 4, 5].map(function factorial(n) { return n <= 1 ? 1 : n * factorial(n - 1); }) }}',
{
__sanitize: sanitizer,
},
);
expect(result).toEqual([1, 2, 6, 24, 120]);
});
it('should handle nested function expression', () => {
const result = tournament.execute(
'{{ [1, 2, 3].map(function(n) { return function() { return n * 2; }; }).map(function(fn) { return fn(); }) }}',
{
__sanitize: sanitizer,
},
);
expect(result).toEqual([2, 4, 6]);
});
it('should handle nested recursion', () => {
const result = tournament.execute(
'{{ (function fibonacci(n) { return n <= 1 ? n : fibonacci(n - 1) + fibonacci(n - 2); })(7) }}',
{
__sanitize: sanitizer,
},
);
expect(result).toBe(13);
});
});
describe('process.env security', () => {
it('should bind function expressions to empty process object', () => {
const processResult = tournament.execute('{{ (function(){ return this.process })() }}', {
__sanitize: sanitizer,
});
expect(processResult).toEqual({});
expect(Object.keys(processResult as object)).toEqual([]);
const envResult = tournament.execute('{{ (function(){ return this.process.env })() }}', {
__sanitize: sanitizer,
});
expect(envResult).toBe(undefined);
});
it('should block process.env in nested functions', () => {
const result = tournament.execute(
'{{ (function outer(){ return (function inner(){ return this.process.env })(); })() }}',
{
__sanitize: sanitizer,
},
);
expect(result).toBe(undefined);
});
it('should block process.env in callbacks', () => {
const result = tournament.execute(
'{{ [1].map(function(){ return this.process.env; })[0] }}',
{
__sanitize: sanitizer,
},
);
expect(result).toBe(undefined);
});
it('should still allow access to workflow variables', () => {
const result = tournament.execute('{{ (function(){ return $json.value })() }}', {
__sanitize: sanitizer,
$json: { value: 'workflow-data' },
});
expect(result).toBe('workflow-data');
});
});
describe('globalThis access via arrow functions', () => {
it('should replace globalThis with empty object', () => {
const result = tournament.execute('{{ (() => globalThis)() }}', {
__sanitize: sanitizer,
});
expect(result).toEqual({});
expect(result).not.toBe(globalThis);
});
it('should block process.env access via globalThis', () => {
const result = tournament.execute('{{ (() => globalThis.process)() }}', {
__sanitize: sanitizer,
});
expect(result).toBe(undefined);
});
it('should block chained globalThis access', () => {
const result = tournament.execute('{{ ((g) => g.process)((() => globalThis)()) }}', {
__sanitize: sanitizer,
});
expect(result).toBe(undefined);
});
it('should block env access via nested arrow functions', () => {
// This payload attempts to access process.env via chained arrow functions
// With the fix, globalThis becomes {}, so g.process is undefined,
// and accessing .env on undefined throws an error - which is the desired security outcome
expect(() => {
tournament.execute('{{ ((p) => p.env)(((g) => g.process)((() => globalThis)())) }}', {
__sanitize: sanitizer,
});
}).toThrow();
});
it('should replace globalThis with empty object in non-arrow contexts too', () => {
// globalThis is replaced with {} at AST level, regardless of context
const result = tournament.execute('{{ globalThis }}', {
__sanitize: sanitizer,
});
expect(result).toEqual({});
});
it('should still allow access to workflow data via variables', () => {
const result = tournament.execute('{{ (() => $json.value)() }}', {
__sanitize: sanitizer,
$json: { value: 'test-value' },
});
expect(result).toBe('test-value');
});
});
describe('this access via arrow functions', () => {
it('should replace this with safe context in arrow functions', () => {
const result = tournament.execute('{{ (() => this)() }}', {
__sanitize: sanitizer,
});
expect(result).toEqual({ process: {}, require: {}, module: {}, Buffer: {} });
});
it('should block process.env access via this in arrow functions', () => {
const result = tournament.execute('{{ (() => this?.process)() }}', {
__sanitize: sanitizer,
});
expect(result).toEqual({});
expect(result).not.toHaveProperty('env');
});
it('should block this access in nested arrow functions', () => {
const result = tournament.execute('{{ (() => (() => this)())() }}', {
__sanitize: sanitizer,
});
expect(result).toEqual({ process: {}, require: {}, module: {}, Buffer: {} });
});
it('should block this?.process?.env access pattern', () => {
const result = tournament.execute('{{ (() => this?.process?.env)() }}', {
__sanitize: sanitizer,
});
expect(result).toBe(undefined);
});
it('should still work with this in regular function expressions', () => {
const result = tournament.execute('{{ (function() { return this.process; })() }}', {
__sanitize: sanitizer,
});
expect(result).toEqual({});
});
});
});