Skip to content
This repository has been archived by the owner on Mar 19, 2021. It is now read-only.

only read 256 kB #2

Open
chadwhitacre opened this issue Sep 10, 2015 · 1 comment
Open

only read 256 kB #2

chadwhitacre opened this issue Sep 10, 2015 · 1 comment

Comments

@chadwhitacre
Copy link
Contributor

The filesize validation is soft and not hard: it's easily gotten around by sending a false Content-Length. We should only read at most the first 256 kB of the upload, and 413 if we don't find EOF before then. This might require some Aspen request.body modifications.

@chadwhitacre
Copy link
Contributor Author

chadwhitacre commented Mar 20, 2017

Doesn't look like we can't configure this at Heroku or in gunicorn. We either need to migrate to Digital Ocean so we can put nginx in front, or implement in gip.rocks itself using Aspen hooks. The latter is probably enough for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant