-
Notifications
You must be signed in to change notification settings - Fork 23
v1.3.0 breaks URL signing #9
Comments
Oh no! Sorry about that, I didn't think of it as a breaking change but I realize now that I changed so the payload argument is now hashed inside the library which may be the issue here (because of different behaviour between services). Could you try just passing an empty string instead of the hash as the 5th argument? I even had an integration test just like yours for this release .. |
Hey it's broken for me as well. I've just tried replacing what I made with that which published and its gone back to the way it was to start with. I am getting no token anymore. That switching in the IoT must be doing it. Anyway I'll use mine until its sorted. |
I think this is breaking a lot of stuff. I'm coming from the NPM package |
This release also broke https://github.com/kmamykin/aws-mqtt. Lost half a days work looking for bugs in my deployment util I finally landed here 😒. I had to lock version 1.2.1 in |
Guys, try this lib: https://github.com/mhart/aws4 |
The solution for me was to use node version: 10.x in AWS lambda |
Hi,
not complaining, but the v1.3.0 break backwards compatibility. My apps AWS IOT function that uses this package broke. Rolling back to version 1.1.0 fixed everything. My mistake to not pin the version, but still.
I don't really know why it broke, but I use your package in the following way:
So I guess something changed in the
createPresignedURL
methodThe text was updated successfully, but these errors were encountered: