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

add posibility unpack buffered archives #6

Closed
wants to merge 2 commits into from
Closed

add posibility unpack buffered archives #6

wants to merge 2 commits into from

Conversation

doomedraven
Copy link
Contributor

i was need to trick it to avoid usage of tempfiles and avoid unneeded usage of IO, read as zip received in email, email parsed and i have attachment as buffer(cuckoomx PR comming)

i was need to trick it to avoid usage of tempfiles and avoid unneeded usage of IO, read as zip received in email, email parsed and i have attachment as buffer(cuckoomx PR comming)
@coveralls
Copy link

Coverage Status

Coverage decreased (-4.3%) to 88.018% when pulling b8c3ffb on doomedraven:patch-1 into 3e0d9e8 on jbremer:master.

@coveralls
Copy link

coveralls commented Nov 21, 2016

Coverage Status

Coverage decreased (-0.1%) to 92.16% when pulling b8c3ffb on doomedraven:patch-1 into 3e0d9e8 on jbremer:master.

@jbremer
Copy link
Member

jbremer commented Nov 21, 2016

Just so we're on the same page, this is the same as something like File(contents='something'), right? Can you also provide some unit tests?

@doomedraven
Copy link
Contributor Author

damn, it's not requires anymore this, as your example works just fine, I just not figurated out what I was able to pass buff directly

@doomedraven
Copy link
Contributor Author

thanks for example, i checked tests and just not saw it

@doomedraven doomedraven deleted the patch-1 branch November 21, 2016 12:01
psrok1 pushed a commit to CERT-Polska/sflock that referenced this pull request Jul 6, 2021
This pull request was closed.
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.

3 participants