Skip to content
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

Jump to bottom

Presigned URL does not consider the server port number #13981

Closed
tzorzo opened this issue Dec 14, 2022 · 0 comments · Fixed by #14464
Closed

Presigned URL does not consider the server port number #13981

tzorzo opened this issue Dec 14, 2022 · 0 comments · Fixed by #14464
Assignees
@dkocher
Labels
bug s3 AWS S3 Protocol Implementation

Comments

@tzorzo
Copy link

tzorzo commented Dec 14, 2022

I can successfully connect to my compatibile S3 server (minio).
But minio by default operates at port 9000 and the presigned url (both for open and copy) ignores the port number causing a Signature Mismatch Exception on my server.
@dkocher dkocher added the s3 AWS S3 Protocol Implementation label Dec 14, 2022
@dkocher dkocher changed the title Presigned url in Compatibile S3 server does not consider the server port number Presigned URL does not consider the server port number Apr 3, 2023
@dkocher dkocher added the bug label Apr 3, 2023
@dkocher dkocher self-assigned this Apr 3, 2023
dkocher added a commit that referenced this issue Apr 3, 2023
@dkocher
Fix #13981.
26b9614
@dkocher dkocher linked a pull request Apr 3, 2023 that will close this issue
Fix #13981. #14464
Merged
ylangisc added a commit that referenced this issue Apr 3, 2023
@ylangisc
Merge pull request #14464 from iterate-ch/bugfix/GH-13981
50975b9 
Fix #13981.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dkocher dkocher

Labels
bug s3 AWS S3 Protocol Implementation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #13981. iterate-ch/cyberduck
2 participants
@dkocher @tzorzo