-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Sigv4a signing region set config bugfix #3279
Sigv4a signing region set config bugfix #3279
Conversation
Fixes a bug where the sigv4a_signing_region_set configuration in the config file and the corresponding environment variable would be ignored
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a functional test that exercises the ConfigProvider to ensure we don't regress this again in the future?
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #3279 +/- ##
========================================
Coverage 93.17% 93.18%
========================================
Files 66 66
Lines 14339 14345 +6
========================================
+ Hits 13361 13367 +6
Misses 978 978 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🏆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Looks like the message
arg in test_auth_config#assert_all_requirements_match
is unused.
Co-authored-by: Nate Prewitt <nate.prewitt@gmail.com>
fb7cd10
to
b03041b
Compare
* release-1.35.42: Bumping version to 1.35.42 Update to latest models Merge customizations for S3 Sigv4a signing region set config bugfix (#3279)
Fixes a bug where the sigv4a_signing_region_set configuration in the config file and the corresponding environment variable would be ignored