Skip to content
This repository was archived by the owner on Jul 19, 2025. It is now read-only.

Conversation

@yigitsoy
Copy link

This PR attempts to address the issue described in #89 where the files can be uploaded to a remote s3 bucket directly without local storage.

@np-8 I tried to keep the changes only on python side. I would be happy if you could see if there are any places that can be improved.

@yigitsoy yigitsoy changed the title Direct upload remote s3 Direct upload to remote s3 May 10, 2022
@fohrloop
Copy link
Owner

fohrloop commented Jul 4, 2022

Hi @yigitsoy ! Thanks for the PR and sorry for the delay! Did you have time to test this with S3? Adding automated test would not hurt, and I guess the test should just check if the send HTTP requests look correct

@yigitsoy
Copy link
Author

yigitsoy commented Jul 8, 2022

Hi @np-8 Thanks for the feedback. I have been using this version to upload to S3 since then. I will try to some tests to make sure that it works in general.

@mapix
Copy link
Contributor

mapix commented Nov 27, 2023

@yigitsoy
Great job, I will assist you in the follow-up development and test this feature together.

@mapix mapix self-requested a review November 27, 2023 06:59
@samkikibaker
Copy link

Very keen to see this as a feature so giving this a bump

@addionisio
Copy link

Would also love to see this feature. Has anyone had issues using this forked version with a Dash app that has a requests_pathname_prefix defined? When I have it defined, the s3 uploader fails to upload but when I remove it, the uploader works perfectly fine.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants