-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Decompression bomb protection #674
Conversation
How is this one coming along? |
@aclark4life It's ready for review. Discussion has been going on over at #515. |
Should I manually merge this? |
I'm not sure that we have consensus on the intended behaviour change. |
Look good to me, though maybe it could use a custom subcass of
It seems innocuous enough that BC proponents would not mind it, and easy enough to notice that more hardline ones would be OK with it (and it's easy enough to turn warnings to errors). |
Conflicts: PIL/Image.py
Change the decompression bomb warning into a subclass of RuntimeWarning to aid filtering
@masklinn Good idea, I've changed the warning to a custom |
Decompression bomb protection
Thanks |
For #515.
Further discussion welcome.