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
Add AWS Signature Version 4 #2036
Conversation
added aws-sign4 support added tests Added sign_version parameter doc fixed test fix changes removed changes on CHANGELOG fix fix fix comma
When this will be merged? |
I'm thinking about moving the require statement down below inside the aws handler and make aws4 external dependency. I don't feel comfortable adding more dependencies in request. Similar to how it's done in request/core. |
Ok feel free to apply your edits, you know what are a good stuff for the module |
2.68 is published 🎉 |
, amazonHeaders: aws.canonicalizeHeaders(self.headers) | ||
|
||
if (opts.sign_version == 4 || opts.sign_version == '4') { | ||
var aws4 = require('aws4') |
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.
This dependency was only added as a devDependency. See #2040
It was moved in request#2036 as a devDependency, but is still used in request.js This should fix request#2040.
Closes #2031
https://www.npmjs.com/package/aws4)