-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Added AWS v4 signature support #23467
Conversation
@slinu3d, thanks for fixing this. Will you also fix these lint errors? Thanks. |
The lint error just does not make sense since there is a space after ',' for sure. |
It's probably complaining about not having a space after the |
Added AWS v4 signature support
@slinu3d Because of new features and changes in the 2015.5 branch, this fix is not merging forward cleanly. Could you please make these same changes in a pull request against the 2015.5 branch? Otherwise this fix will not continue into new releases of salt. |
Created PR (#23547) for 2015.5. |
Thanks @slinu3d. |
Thanks! |
This patch adds AWS v4 signature support and fixes issue #20518