#20331 closed Bug (fixed)
Django admin action cannot serve StreamingHttpResponse
| Reported by: | Owned by: | nobody | |
|---|---|---|---|
| Component: | contrib.admin | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | timograham@… | Triage Stage: | Ready for checkin |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
I think it's reasonable enough to have an admin action that serves a StreamingHttpResponse.
Patch is attached.
Attachments (2)
Change History (14)
by , 13 years ago
| Attachment: | admin-action-streaming.patch added |
|---|
comment:1 by , 13 years ago
| Component: | Uncategorized → contrib.admin |
|---|---|
| Has patch: | set |
| Type: | Uncategorized → Bug |
comment:2 by , 13 years ago
| Needs documentation: | set |
|---|---|
| Needs tests: | set |
| Triage Stage: | Unreviewed → Accepted |
| Version: | 1.5 → master |
comment:3 by , 13 years ago
Yay type checks :-/
The proper solution is to test for HttpResponseBase (a private API that lives in django.http.response).
comment:4 by , 13 years ago
| Patch needs improvement: | set |
|---|
by , 13 years ago
| Attachment: | 20331.diff added |
|---|
comment:5 by , 13 years ago
| Needs documentation: | unset |
|---|---|
| Needs tests: | unset |
| Patch needs improvement: | unset |
| Triage Stage: | Accepted → Ready for checkin |
Patch looks pretty good
comment:6 by , 13 years ago
I'm committing this, I'll just remove the line from the release notes because it's really a bugfix, and we don't mention them all.
comment:7 by , 13 years ago
| Patch needs improvement: | set |
|---|---|
| Triage Stage: | Ready for checkin → Accepted |
Hmm, actually six admin_views tests fail under Python 3 with this patch.
comment:8 by , 13 years ago
| Cc: | added |
|---|---|
| Patch needs improvement: | unset |
Fixed tests and removed release note.
comment:9 by , 13 years ago
| Triage Stage: | Accepted → Ready for checkin |
|---|
Ran all admin_views tests on Python 2.7.3 and 3.2.3 with success, marking as RFC.
comment:10 by , 13 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:11 by , 11 years ago
Is there any policy on backporting this fix to older versions...? my project is on 1.5 and I just hit this bug
I'm happy to do the work if it can get merged quickly
comment:12 by , 11 years ago
1.5 is receiving only security fixes at this time. See the supported versions policy for details.
This is a reasonable request indeed. This needs testing and an entry in the release note.