Daniel-t / node-red-contrib-aws

A collection of Node-RED nodes for AWS.
Apache License 2.0
56 stars 59 forks source link

DynamoDB errors lose original message #61

Open bewood opened 4 years ago

bewood commented 4 years ago

Expected

The error output of a DynamoDB node to add msg.err to existing msg object.

Actual

The error output of a DynamoDB node is a new msg with only msg.err and a new msg._msgid property.

Impact

This causes issues using DynamoDB in an HTTP endpoint flow because it loses the msg.res object. Additionally, mitigations by stashing msg.res in flow context aren't possible because even the msg._msgid is lost.

thomaszlow commented 1 year ago

I'm facing the same difficulties because of the missing msgobject.

quarfie commented 1 year ago

This affects all nodes, not just DynamoDB. It is a problem for me, too, as I run these nodes from flows that are exposed as web services. Without the msg.res and msg.req data in the error, it is impossible to send a response to the original http request, so it must time out instead.