Skip to content
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

aws-sdk-lambda fix #2939

Merged
merged 1 commit into from
Nov 7, 2024
Merged

aws-sdk-lambda fix #2939

merged 1 commit into from
Nov 7, 2024

Conversation

fallwith
Copy link
Contributor

@fallwith fallwith commented Nov 6, 2024

commit 0a1527d sought to improve error handling, but incorrectly misused an ensure block to attempt to always return the raw client response.

this faulty logic has been fixed. internal agent response handling errors are logged as desired, not re-raised to the user's app, and the user's app is guaranteed to always get the raw original response.

commit 0a1527d sought to improve error
handling, but incorrectly misused an `ensure` block to attempt to always
return the raw client response.

this faulty logic has been fixed. internal agent response handling
errors are logged as desired, not re-raised to the user's app, and the
user's app is guaranteed to always get the raw original response.
Copy link
Contributor

github-actions bot commented Nov 6, 2024

SimpleCov Report

Coverage Threshold
Line 93.85% 93%
Branch 69.73% 50%

@fallwith fallwith merged commit 1d70cad into dev Nov 7, 2024
36 of 37 checks passed
@fallwith fallwith deleted the half-life branch November 7, 2024 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants