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

Raise presigned URL timeout to 24 hours for multipart uploads #85

Merged
merged 1 commit into from
Jul 11, 2023

Conversation

liam-lloyd
Copy link
Member

Historically, our presigned URLs for uploads have expired after 1 hour. This makes sense for the uploads we've been doing, where the presigned URL would be hit near instantly after it was created, but in a large multipart upload it may be some time before each URL has been called. To address this, this commit raises the timeout for presigned URLs involved in a multipart upload to 24 hours.

Historically, our presigned URLs for uploads have expired after 1 hour.
This makes sense for the uploads we've been doing, where the presigned
URL would be hit near instantly after it was created, but in a large
multipart upload it may be some time before each URL has been called. To
address this, this commit raises the timeout for presigned URLs involved
in a multipart upload to 24 hours.
Copy link
Member

@cecilia-donnelly cecilia-donnelly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't test this, but it seems perfectly reasonable

@liam-lloyd liam-lloyd merged commit 8ab8237 into main Jul 11, 2023
@liam-lloyd liam-lloyd deleted the raise-presigned-url-timeout-for-multipart-upload branch July 11, 2023 18:12
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.

2 participants