add error replacement for DynamoDBStreams v2 #11887
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
As reported by a user, we had a red herring regarding some raised DynamoDB Streams error when used with ESM (with non default account and region):
Which made it look like we were not properly formatting the ARN or something related to DDB Local.
The issue was that we were not replacing the ARN value in the exception as it was directly raised by DDB-local, but our headers were correct for the namespacing.
Also, made a quick fix to start DDB-local in DynamoDB Streams v2 because if you targeted the
dynamodbstreams
service before doing a request todynamodb
, you would get a connection refused error as we'd try to send a request to it. I'm not adding all the hooks related to persistence because those services are very intertwined and this would only happen at startup with a fresh instance.Changes