Opened 11 months ago
Last modified 4 months ago
#35281 assigned Cleanup/optimization
Use HTTP 413 status code for RequestDataTooBig
Description ¶
Other HTTP servers (e.g. Apache, Nginx) report a body which is to large with the status code 413 "Payload Too Large".
I propose to do the same in Django when handling a RequestDataTooBig exception in https://github.com/django/django/blob/f931cbec21cce3e2e68239c77614e6aa5026b882/django/core/handlers/exception.py#L63
This makes the behavior of a typical stack consisting of a reverse proxy and a Django project predictable for this type of error.
I can work on a patch if this change is acceptable to you.
According to the ticket's flags, the next step(s) to move this issue forward are:
- To provide a patch by sending a pull request. Claim the ticket when you start working so that someone else doesn't duplicate effort. Before sending a pull request, review your work against the patch review checklist. Check the "Has patch" flag on the ticket after sending a pull request and include a link to the pull request in the ticket comment when making that update. The usual format is:
[https://github.com/django/django/pull/#### PR]
.
Change History (13)
comment:1 by , 11 months ago
Triage Stage: | Unreviewed → Accepted |
---|---|
Type: | Uncategorized → Cleanup/optimization |
comment:2 by , 11 months ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:3 by , 10 months ago
Hello,
Is this issue still active?
I'm new to both Django and contributing to open-source, and would be more than excited to work on a patch that fixes this.
comment:4 by , 10 months ago
Feel free to take over! Also note this comment: https://github.com/django/django/pull/17960#pullrequestreview-1926197306
comment:5 by , 10 months ago
Owner: | removed |
---|---|
Status: | assigned → new |
comment:6 by , 10 months ago
Owner: | set to |
---|---|
Status: | new → assigned |
comment:7 by , 9 months ago
Hello, I noticed that there hasn't been a modification in 4 weeks. Is it ok if I take over? I am a first time contributor to django and would love the opportunity to take a crack at this.
follow-up: 9 comment:8 by , 9 months ago
Easy pickings: | unset |
---|---|
Owner: | removed |
Status: | assigned → new |
Robin feel free to assign but take a minute to digest why the efforts have stalled in the past weeks first. I don't this this constitutes an easy picking anymore.
Milan, I'll unassign given you closed your MR. Thank you for kicking off the work.
follow-up: 10 comment:9 by , 6 months ago
Replying to Simon Charette:
Robin feel free to assign but take a minute to digest why the efforts have stalled in the past weeks first. I don't this this constitutes an easy picking anymore.
Milan, I'll unassign given you closed your MR. Thank you for kicking off the work.
Can I handle this one? I have the time needed
comment:10 by , 6 months ago
Replying to Ahmed Ibrahim:
Replying to Simon Charette:
Robin feel free to assign but take a minute to digest why the efforts have stalled in the past weeks first. I don't this this constitutes an easy picking anymore.
Milan, I'll unassign given you closed your MR. Thank you for kicking off the work.
Can I handle this one? I have the time needed
Hello Ahmed, thank you for your interest in helping Django. Yes, you can assign yourself the ticket and work on it. Please take into consideration all the commentary, specifically the one from the closed PR.
comment:12 by , 5 months ago
Cc: | added |
---|
comment:13 by , 4 months ago
Note that I provided a WIP PR to explore adding a generic error view which should ease going forward with this ticket. It's a POC for feedback, but not in a reviewable state.
Following https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/413 I think the request makes sense.
I haven't found a previous report about this and the code that added some handling for
RequestDataTooBig
may have missed to return a more specific code. Tangential follow up of #27820.