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

Handle yauzl Zipfile 'error' emit #68

Merged
merged 1 commit into from
Dec 4, 2015

Conversation

ahgentil
Copy link
Contributor

@ahgentil ahgentil commented Dec 4, 2015

If yauzl fails while extracting the content of the zipfile, it emits 'error', and there's no handling at all if that occurs on textract.
I faced the problem when I passed a corrupted .docx file to textract and the server 'blew up' :).
This was a quick fix. Maybe I'm missing other stuff.

dbashford added a commit that referenced this pull request Dec 4, 2015
Handle yauzl Zipfile 'error' emit
@dbashford dbashford merged commit 7d35e16 into dbashford:master Dec 4, 2015
@dbashford
Copy link
Owner

Thanks!

Released as 1.2.1.

@ahgentil
Copy link
Contributor Author

ahgentil commented Dec 4, 2015

👍

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