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

Suppress PIL.Image.DecompressionBombError #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

peeves91
Copy link

@peeves91 peeves91 commented Mar 6, 2019

this error is thrown by PIL when you put an exceptionally large image into it, since it thinks it could be a DOS attack, and it rejects it. adding this line prevents such errors.

@Menotdan
Copy link

Yep I had this problem too.

@mikepound
Copy link
Owner

Sorry for only just getting around to this! This looks good, could you alter the commit to re-include the print("Loading Image") line before I merge?

Copy link

@mschultheiss83 mschultheiss83 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add print ("Loading Image")

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.

4 participants