mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2024-12-26 05:04:05 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

refactor(Code Node): Limit n8n item key check (#4737)

Browse source 
⚡ Limit n8n item key check
This commit is contained in:
Iván Ovejero 2022-12-05 10:59:26 +01:00 committed by GitHub
parent 697efb2d31
commit 46d4c4d329
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
packages/nodes-base/nodes/Code/Sandbox.ts
View file

@ -2,7 +2,7 @@ import { normalizeItems } from 'n8n-core';
import { NodeVM, NodeVMOptions } from 'vm2'; import { NodeVM, NodeVMOptions } from 'vm2';
import { ValidationError } from './ValidationError'; import { ValidationError } from './ValidationError';
import { ExecutionError } from './ExecutionError'; import { ExecutionError } from './ExecutionError';
import { CodeNodeMode, isObject, N8N_ITEM_KEYS } from './utils'; import { CodeNodeMode, isObject, REQUIRED_N8N_ITEM_KEYS } from './utils';
import type { IExecuteFunctions, IWorkflowDataProxyData, WorkflowExecuteMode } from 'n8n-workflow'; import type { IExecuteFunctions, IWorkflowDataProxyData, WorkflowExecuteMode } from 'n8n-workflow';
@ -104,7 +104,7 @@ export class Sandbox extends NodeVM {
* item keys to be wrapped in `json` when normalizing items below. * item keys to be wrapped in `json` when normalizing items below.
*/ */
const mustHaveTopLevelN8nKey = executionResult.some((item) => const mustHaveTopLevelN8nKey = executionResult.some((item) =>
Object.keys(item).find((key) => N8N_ITEM_KEYS.has(key)), Object.keys(item).find((key) => REQUIRED_N8N_ITEM_KEYS.has(key)),
); );
for (const item of executionResult) { for (const item of executionResult) {
@ -118,7 +118,7 @@ export class Sandbox extends NodeVM {
if (mustHaveTopLevelN8nKey) { if (mustHaveTopLevelN8nKey) {
Object.keys(item).forEach((key) => { Object.keys(item).forEach((key) => {
if (N8N_ITEM_KEYS.has(key)) return; if (REQUIRED_N8N_ITEM_KEYS.has(key)) return;
throw new ValidationError({ throw new ValidationError({
message: `Unknown top-level item key: ${key}`, message: `Unknown top-level item key: ${key}`,
description: 'Access the properties of an item under `.json`, e.g. `item.json`', description: 'Access the properties of an item under `.json`, e.g. `item.json`',
@ -226,7 +226,7 @@ export class Sandbox extends NodeVM {
// directly on the item, when they intended to add it on the `json` property // directly on the item, when they intended to add it on the `json` property
Object.keys(executionResult).forEach((key) => { Object.keys(executionResult).forEach((key) => {
if (N8N_ITEM_KEYS.has(key)) return; if (REQUIRED_N8N_ITEM_KEYS.has(key)) return;
throw new ValidationError({ throw new ValidationError({
message: `Unknown top-level item key: ${key}`, message: `Unknown top-level item key: ${key}`,

2
packages/nodes-base/nodes/Code/utils.ts
View file

@ -26,4 +26,4 @@ function isTraversable(maybe: unknown): maybe is IDataObject {
export type CodeNodeMode = 'runOnceForAllItems' | 'runOnceForEachItem'; export type CodeNodeMode = 'runOnceForAllItems' | 'runOnceForEachItem';
export const N8N_ITEM_KEYS = new Set(['json', 'binary', 'error', 'pairedItem', 'index']); export const REQUIRED_N8N_ITEM_KEYS = new Set(['json', 'binary']);