-
Notifications
You must be signed in to change notification settings - Fork 34
Include insecure option and tls options when STS S3 token is enabled #1108
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
Conversation
Signed-off-by: Ruben Vargas <[email protected]>
9454df7
to
e5ecce1
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1108 +/- ##
=======================================
Coverage 58.60% 58.60%
=======================================
Files 113 113
Lines 10160 10160
=======================================
Hits 5954 5954
Misses 3900 3900
Partials 306 306
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@IshwarKanse Could you please help me to test this? Thanks |
@rubenvp8510 Sure, I'll check and let you know. |
@rubenvp8510 Verified the bug fix, we are good to merge the PR :) |
Co-authored-by: Andreas Gerstmayr <[email protected]>
Thank you! |
Signed-off-by: Ruben Vargas <[email protected]>
No description provided.