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

Fix presigned url to include port #30

Closed
wants to merge 1 commit into from
Closed

Fix presigned url to include port #30

wants to merge 1 commit into from

Conversation

gperdomor
Copy link

Fix to able to use presigned url with minio

Fix to able to use presigned url with minio
@gperdomor gperdomor changed the title Fix resigned url to include port Fix presigned url to include port Nov 16, 2018
@ldmarz
Copy link

ldmarz commented Nov 16, 2018

Thanks dude I need this to work with PresignedSign using minio

@Joebayld
Copy link
Collaborator

Take a look at this PR. #29

I made a handful of improvements and we might need to tweak it a bit to allow ports. Just a heads up.

@gperdomor gperdomor closed this Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants