-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
"The request signature we calculated does not match the signature you provided." when uploading with Aws::S3::Object#upload_file
#965
Comments
We're taking a look at this, may not be related to the previous issue. |
I'd recommend, if possible, using Signature Version 4 signing in these cases and seeing if that helps to resolve the issue. We're working right now on a 2.2.0 release that will make this the default behavior, but you can also do this right now: client = Aws::S3::Client.new(signature_version: 'v4') |
I'd strongly suspect that our most recent release (2.2.0) will resolve this problem. Can you confirm? |
@awood45 Funny you should mention! I just updated https://github.com/travis-ci/travis-ci/issues/4776. Unfortunately, I've had a few reports (from our customers using private repositories, so I don't have direct links) that 2.2.0 is breaking their deployment with S3. I haven't been able to reproduce this myself, but in one case forcing 2.1.36 fixes the problem. |
Do you have any information about those breakages? |
Let me know if there's any information you can share - I haven't heard of breakages yet, would love to jump on any issues people are seeing immediately. |
@BanzaiMan do we have any updates on 2.2.x helping this issue? I know in the other thread we seemed to be near a resolution to the Python packages issue, but have other customers seen this issue resolve with 2.2.x? |
@awood45 Thanks for the update. Not sure yet whether 2.2.2 or 2.2.3 is helping yet. I also tested on Python, but couldn't reproduce, either. |
@BanzaiMan should I leave this open, or do we think this has brought a resolution to the problem? |
@awood45 Sorry for the delay in update. I'll ask users to see where we are. I haven't noticed any recent report of failures, but I'd like to be certain. |
@BanzaiMan - It's looking like this update has likely fixed the issue, and that this is just a hard one to confirm (understandably). I'm going to close this for now, but I'm still monitoring the TravisCI issue, and please feel free to reopen or to open a new issue if there is any sign that this issue persists (or has recurred in yet another form). |
This is possibly a variation of #923. With 2.1.30, a user reports that uploading file results in signature mismatch. The back traces are almost identical:
Before:
After:
The text was updated successfully, but these errors were encountered: