Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

S3 signature doesn't match expected when path contains special chars #374

Closed
gipeshka opened this issue Jul 3, 2017 · 4 comments · Fixed by #376
Closed

S3 signature doesn't match expected when path contains special chars #374

gipeshka opened this issue Jul 3, 2017 · 4 comments · Fixed by #376
Labels
Milestone

Comments

@gipeshka
Copy link
Contributor

gipeshka commented Jul 3, 2017

Hi guys.

Currently I'm facing an issue with alpakka s3 client 0.10 when accessing a file like files/überall.csv

The request signature we calculated does not match the signature you provided. Check your key and signing method.

When I pass the key with encoded special chars, e.g. files/%C3%BCberall.csv I get another exception: The specified key does not exist.

Supposedly the issue is related to #357 and #316 because in version 0.9 I had exactly the error
akka.http.scaladsl.model.IllegalUriException: Illegal URI reference: Invalid input 'ü', expected '/', 'EOI', '#', '?' or pchar for exactly same configuration.

@francisdb
Copy link
Contributor

Will create a test and look into this!

@gipeshka
Copy link
Contributor Author

gipeshka commented Jul 3, 2017

Hey @francisdb I added one on my local branch, actually shows that it's broken. I'll try to look into it myself, but anyway hope it helps gipeshka@d10058b

@francisdb
Copy link
Contributor

Also got this reproduced, I wonder if there is still some issue with the CanonicalRequest logic

@francisdb
Copy link
Contributor

found the issue, non-ASCII chars are not correctly encoded by CanonicalRequest, fix on it's way

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants