-
Notifications
You must be signed in to change notification settings - Fork 37
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
Zipfile #332
Zipfile #332
Conversation
I'm wondering what the utility of this is, unzipping things can have huge memory consequences. I don't think Larky is the right tool for this. I think we might have some other projects that would be better at unzipping (like SFTP proxy) |
Also needs Jira links so I comment there why this is a Bad Idea |
This is for a TCB project, can explain in Jira: https://verygoodsecurity.atlassian.net/browse/INT-316 |
Commented in the Jira. |
http proxy can unzip file. |
Do we have docs for those? I can't find anything for the FCO's that start with |
I know OMF uses it, maybe you can copy that over to HTTP? |
We've already tried using that operation, it doesn't work on the HTTP Proxy. CC: @alejohern |
Add ZipFile
Description of changes in release / Impact of release:
Adds stdlib
zipfile
functionality to Larky, allowing us to zip/unzip traffic with Larky.Documentation
(insert text here)
Risks of this release
n/a
Is this a breaking change?
If you answered Yes then describe why is it so
(insert text here if applicable)
Is there a way to disable the change?
Additional details go here
(insert text here if applicable)