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

Increase maximum images size #25

Open
3 tasks
kba opened this issue Jan 13, 2022 · 0 comments
Open
3 tasks

Increase maximum images size #25

kba opened this issue Jan 13, 2022 · 0 comments
Labels

Comments

@kba
Copy link
Member

kba commented Jan 13, 2022

Current situation

The upper limit of PIL, our image library, for loading images is 89478485 which is equivalent to ~ 9400x9400 pixels. Large or very high pixel-depth scans, such as of newspapers, maps etc. can be larger than that. Loading a larger image will result in an error to the user.

How it should be

The upper limit should be future-proof, 40k by 40k pixels for a possible image seems a reasonable upper bound.

As we move towards networked components, this setting should be documented to make developers aware that this could be a potential source for DOS attacks by spamming a server with bogus large images.

Steps

@kba kba added the Epic label Jan 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant