#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 , 12 years ago
Attachment: | admin-action-streaming.patch added |
---|
comment:1 by , 12 years ago
Component: | Uncategorized → contrib.admin |
---|---|
Has patch: | set |
Type: | Uncategorized → Bug |
comment:2 by , 12 years ago
Needs documentation: | set |
---|---|
Needs tests: | set |
Triage Stage: | Unreviewed → Accepted |
Version: | 1.5 → master |
comment:3 by , 12 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 , 12 years ago
Patch needs improvement: | set |
---|
by , 12 years ago
Attachment: | 20331.diff added |
---|
comment:5 by , 11 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 , 11 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 , 11 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 , 11 years ago
Cc: | added |
---|---|
Patch needs improvement: | unset |
Fixed tests and removed release note.
comment:9 by , 11 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 , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:11 by , 10 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 , 10 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.