mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-03-05 20:50:17 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix(AWS DynamoDB Node): Add missing type assertion (no-changelog) (#5003)

Browse source 
fix(AWS DynamoDB Node): add type assertion
This commit is contained in:
Csaba Tuncsik 2022-12-22 10:27:14 +01:00 committed by GitHub
parent 87d8865ad3
commit e4785da2e1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 25 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
packages/nodes-base/nodes/Aws/DynamoDB/utils.ts
View file

@ -1,4 +1,4 @@
import { deepCopy, IDataObject, INodeExecutionData } from 'n8n-workflow'; import { deepCopy, IDataObject, INodeExecutionData, assert } from 'n8n-workflow';
import { import {
AdjustedPutItem, AdjustedPutItem,
@ -82,7 +82,11 @@ function decodeAttribute(type: AttributeValueType, attribute: string | IAttribut
case 'NS': case 'NS':
return attribute; return attribute;
case 'M': case 'M':
return simplify(attribute as IAttributeValue); assert(
typeof attribute === 'object' && !Array.isArray(attribute) && attribute !== null,
'Attribute must be an object',
);
return simplify(attribute);
default: default:
return null; return null;
} }

2
packages/workflow/src/index.ts
View file

@ -18,7 +18,7 @@ export * from './WorkflowErrors';
export * from './WorkflowHooks'; export * from './WorkflowHooks';
export * from './VersionedNodeType'; export * from './VersionedNodeType';
export { LoggerProxy, NodeHelpers, ObservableObject, TelemetryHelpers }; export { LoggerProxy, NodeHelpers, ObservableObject, TelemetryHelpers };
export { deepCopy, jsonParse, sleep, fileTypeFromMimeType } from './utils'; export { deepCopy, jsonParse, sleep, fileTypeFromMimeType, assert } from './utils';
export { export {
isINodeProperties, isINodeProperties,
isINodePropertyOptions, isINodePropertyOptions,

18
packages/workflow/src/utils.ts
View file

@ -74,3 +74,21 @@ export function fileTypeFromMimeType(mimeType: string): BinaryFileType | undefin
if (mimeType.startsWith('text/')) return 'text'; if (mimeType.startsWith('text/')) return 'text';
return; return;
} }
export function assert<T>(condition: T, msg?: string): asserts condition {
if (!condition) {
const error = new Error(msg ?? 'Invalid assertion');
// hide assert stack frame if supported
if (Error.hasOwnProperty('captureStackTrace')) {
// V8 only - https://nodejs.org/api/errors.html#errors_error_capturestacktrace_targetobject_constructoropt
Error.captureStackTrace(error, assert);
} else if (error.stack) {
// fallback for IE and Firefox
error.stack = error.stack
.split('\n')
.slice(1) // skip assert function from stack frames
.join('\n');
}
throw error;
}
}