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

Remove unnecessary .env vars from setup instructions #15

Merged
merged 1 commit into from
Jan 5, 2023

Conversation

kerkness
Copy link
Contributor

@kerkness kerkness commented Jan 4, 2023

AWS_URL and AWS_POST_END_POINT environment variables are now dynamically determined by the aws-sdk-php package. Setting these to any value will make the sdk automatically assume you are using a custom endpoint and prevent the use of accelerator,

Fixes #14

`AWS_URL` and `AWS_POST_END_POINT` environment variables are now dynamically determined by the aws-sdk-php package.  Setting these to any value will make the sdk automatically assume you are using a custom endpoint and prevent the use of accelerator,
@swilla swilla requested a review from andreia January 5, 2023 00:41
@andreia
Copy link
Member

andreia commented Jan 5, 2023

Thank you @kerkness for this PR and for taking the time to investigate this!

@andreia andreia merged commit 12d82a1 into TappNetwork:master Jan 5, 2023
@stobrien89
Copy link

stobrien89 commented Jan 5, 2023

@andreia, @kerkness

FWIW it might be worth adding those back and just mentioning that they are optional, i.e. should only be used if a customer is using a custom, non-aws endpoint. I could see the lack of documentation for those two env vars causing confusion for those who may be using a 3rd party object store. Definitely not the common case, but it's something to keep in mind.

@andreia
Copy link
Member

andreia commented Jan 7, 2023

@stobrien89 @kerkness Yeah, added a note in #16

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.

createMultipartUpload throwing aws Host override error
3 participants