-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(stepfunctions): repeated object references not allowed even if not a circular reference #14628
fix(stepfunctions): repeated object references not allowed even if not a circular reference #14628
Conversation
…be allowed if not causing a circular reference fixes aws#14596
Pull request has been modified.
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
…t a circular reference (aws#14628) Popping the current node from `visited` array after finishing visiting it in `recurseObject` method. This allows another reference to the same object to happen on a different object path (currently being swallowed). Created unit test before making any code changes to verify the failure scenario on the expected behavior before fixing it. fixes aws#14596 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Popping the current node from
visited
array after finishing visiting it inrecurseObject
method. This allows another reference to the same object to happen on a different object path (currently being swallowed).Created unit test before making any code changes to verify the failure scenario on the expected behavior before fixing it.
fixes #14596
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license