github-codebuild-logs icon indicating copy to clipboard operation
github-codebuild-logs copied to clipboard

S3 presigned URLs are sometimes broken

Open mattsb42 opened this issue 5 years ago • 11 comments

I haven't started digging to see what is causing this yet, but some of the presigned URLs appear to be calculating incorrectly.

ex: https://github.com/mattsb42/pipipxx/pull/12

mattsb42 avatar Oct 19 '19 12:10 mattsb42

@mattsb42 I clicked on the 3 build logs links in the example PR a few times, and they all loaded for me. Are you still seeing this issue? When you see it, what happens? Is there an error?

jlhood avatar Oct 21 '19 18:10 jlhood

weird; they're working for me now too...I'll see if I can replicate the behavior I saw

mattsb42 avatar Oct 21 '19 19:10 mattsb42

k, I'll go ahead and close this for now. Feel free to reopen if you're able to reproduce it consistently.

jlhood avatar Oct 21 '19 20:10 jlhood

Pinging this issue to say that I experienced the same thing (but can no longer repro). After initially setting up the app, the links posted to PRs had some sort of signing issue. Unfortunately I did not save the exact error, but it looks like the same one @mattsb42 described above.

A day later, I try the links again and they all work.

lavaleri avatar Nov 22 '19 19:11 lavaleri

I probably have the same issue. The error message is like this <Error> <Code>SignatureDoesNotMatch</Code> <Message> The request signature we calculated does not match the signature you provided. Check your key and signing method. </Message>

RayMin920 avatar Jan 08 '20 22:01 RayMin920

I am getting the same issue too:

<Error>
<Code>SignatureDoesNotMatch</Code>
<Message>
The request signature we calculated does not match the signature you provided. Check your key and signing method.
</Message>

Does any one have a solution?

jeansymolanza avatar May 24 '20 13:05 jeansymolanza

I probably have the same issue. The error message is like this <Error> <Code>SignatureDoesNotMatch</Code> <Message> The request signature we calculated does not match the signature you provided. Check your key and signing method. </Message>

@mattsb42 @RayMin920 did you ever get the issue resolved?

jeansymolanza avatar May 24 '20 15:05 jeansymolanza

It works now, I guess there is a delay between initial setup and the S3 pre-signed URLs working.

jeansymolanza avatar May 25 '20 09:05 jeansymolanza

is this issue fixed? I'm getting the same issue while browsing the build log endpoint.

dh-shreeram avatar Oct 29 '20 07:10 dh-shreeram

@jlhood is there any update on this issue? I'm getting the same issue , tried to setup multiple times but got the same issue while browsing the build-log endpoint.

dh-shreeram avatar Oct 30 '20 05:10 dh-shreeram

We also had the same issue in our repository immediately after setting up the app yesterday: we would receive comments indicating a successful or failing build but the logs couldn't be viewed due to a SignatureDoesNotMatch error. It seems to have resolved itself after 24 hours.

dominic-mulligan-arm avatar Nov 10 '20 14:11 dominic-mulligan-arm