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

createMultipartUpload throwing aws Host override error #14

Closed
kerkness opened this issue Jan 1, 2023 · 1 comment · Fixed by #15
Closed

createMultipartUpload throwing aws Host override error #14

kerkness opened this issue Jan 1, 2023 · 1 comment · Fixed by #15

Comments

@kerkness
Copy link
Contributor

kerkness commented Jan 1, 2023

A new issue popped up after doing a fresh deploy which appears to result from updates with the AWS SDK but unclear to me if this new error is on the Uppy side of things, the laravel side of things or with AWS itself.

The call to $this->client->createMultipartUpload in UppyS3MultipartController (line 103) is throwing the error Host override cannot be combined with Dualstack, FIPS, or S3 Accelerate from AWS.

There doesn't appear to be a lot of details on this error. I'm curious if others are getting it as my uploading has been very stable for many months with no changes.

Also, the exception being thrown does not support $exception->getStatusCode() which needs to be removed from the try/catch in order to see the actual error from AWS.

@kerkness
Copy link
Contributor Author

kerkness commented Jan 4, 2023

As detailed here aws/aws-sdk-php#2614

Recent the aws-sdk-php package will now dynamically determine the necessary endpoint for accessing S3. If any value is set in AWS_URL or AWS_POST_END_POINT then aws-sdk-php will consider this a custom endpoint even if the value set is not in-fact a custom endpoint.

Therefore the documentation instructions to set these environment variables will cause issues if you use accelerator with your bucket. They do not need to be set with the lastest versions of the aws sdk.

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 a pull request may close this issue.

1 participant