-
Notifications
You must be signed in to change notification settings - Fork 42
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
Add multi-file write support #196
base: dev
Are you sure you want to change the base?
Add multi-file write support #196
Conversation
…l-files-to-a-sandbox-at-once-e2b-954
@ValentaTomas per discussion in e2b-dev/E2B#451, I overloaded the |
Yes, I think it should be okay for us because we are giving users access to the whole sandbox, so this is not a security risk. |
Is it tested for the old file uploads via SDK when passing relative and absolute path and uploads via URL (with the path being part of the multipart)? |
…y-to-upload-several-files-to-a-sandbox-at-once-e2b-954
New write tests have passed as follow: https://github.com/e2b-dev/E2B/pull/503/files#diff-7a70570a2ea87cb7d3e355b0aa6f011a4bec4d1733f40bf94b05e4024f232b01 |
Description
Create separate
multipart.Part
method to get file name with full path with directories.Tests
See the tests in e2b-dev/E2B#451