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] S3 uploads not working for custom URLs #7443

Merged
merged 2 commits into from
Jul 14, 2017

Conversation

rodrigok
Copy link
Member

@rodrigok rodrigok commented Jul 6, 2017

Closes #7411

@rodrigok rodrigok added this to the 0.57.2 milestone Jul 6, 2017
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7443 July 6, 2017 19:27 Inactive
@rodrigok rodrigok force-pushed the hotfix/allow-s3-custom-endpoint branch from 5e08965 to 2a8ea73 Compare July 6, 2017 19:28
@rodrigok rodrigok temporarily deployed to rocket-chat-pr-7443 July 6, 2017 19:28 Inactive
@rodrigok rodrigok requested a review from sampaiodiego July 6, 2017 19:29
@TheReal1604
Copy link
Contributor

@rodrigok Thats awesome, thanks for the fast fix!

@sampaiodiego sampaiodiego changed the title [FIX] S3 uplaods not working for custom URLs [FIX] S3 uploads not working for custom URLs Jul 7, 2017
@geekgonecrazy
Copy link
Contributor

On minio I now get this:
image

@sampaiodiego
Copy link
Member

which version @geekgonecrazy ? @rodrigok tested it, but I don't which version he used

@rodrigok
Copy link
Member Author

@geekgonecrazy I added more 2 options to configure s3, if you are running minio without ssl you need to configure the Signature Version to v2 and enable Force Path Style

@geekgonecrazy
Copy link
Contributor

which version @geekgonecrazy ? @rodrigok tested it, but I don't which version he used
@sampaiodiego the latest docker image is how I ran it.

I didn't use SSL. I'll try again with these changes and see if that fixes it 👍

@geekgonecrazy
Copy link
Contributor

Switching to v2 and Force Path Style worked i'll update document

@rodrigok rodrigok merged commit 291cb73 into develop Jul 14, 2017
@rodrigok rodrigok deleted the hotfix/allow-s3-custom-endpoint branch July 14, 2017 18:21
rodrigok added a commit that referenced this pull request Jul 14, 2017
[FIX] S3 uploads not working for custom URLs
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.

Minio support as AWS s3 storage broken
5 participants