#7936 closed New feature (fixed)
Add Last-Modified header to feeds
Reported by: | Julian Bez | Owned by: | |
---|---|---|---|
Component: | contrib.syndication | Version: | master |
Severity: | Normal | Keywords: | syndication last-modified |
Cc: | tom@…, d1fffuz0r@… | Triage Stage: | Accepted |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | yes |
Easy pickings: | no | UI/UX: | no |
Description
The patch adds a Last-Modified header to feeds generated by the syndication app.
We already have the latest date, so why not use it?
Works best with django.middleware.http.ConditionalGetMiddleware, which does the work of responding with 304 etc.
Attachments (2)
Change History (22)
Changed 11 years ago by
Attachment: | syndication-lastmodified.diff added |
---|
comment:1 Changed 11 years ago by
Status: | new → assigned |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:2 Changed 11 years ago by
Component: | Contrib apps → RSS framework |
---|
comment:3 Changed 11 years ago by
milestone: | → 1.0 |
---|
comment:4 Changed 11 years ago by
milestone: | 1.0 → post-1.0 |
---|
comment:5 Changed 10 years ago by
Version: | SVN → 1.0 |
---|
comment:8 Changed 9 years ago by
Version: | 1.0 → 1.2-alpha |
---|
comment:9 Changed 9 years ago by
milestone: | → 1.2 |
---|
comment:10 Changed 9 years ago by
milestone: | 1.2 |
---|
This is a feature request, and so it cannot go on the 1.2 milestone; 1.2 is feature frozen.
comment:11 Changed 9 years ago by
Owner: | Julian Bez deleted |
---|---|
Status: | assigned → new |
Changed 9 years ago by
Attachment: | syndication-lastmodified.2.diff added |
---|
Patch updated against 1.2.1
comment:13 Changed 8 years ago by
Severity: | → Normal |
---|---|
Type: | → New feature |
comment:14 Changed 8 years ago by
Cc: | tom@… added |
---|---|
Easy pickings: | unset |
comment:15 Changed 7 years ago by
Patch needs improvement: | set |
---|---|
UI/UX: | unset |
Patch no longer applies.
comment:16 Changed 6 years ago by
Cc: | d1fffuz0r@… added |
---|---|
Has patch: | unset |
Needs tests: | unset |
Version: | 1.2-alpha → master |
comment:17 Changed 6 years ago by
Owner: | set to Claude Paroz <claude@…> |
---|---|
Resolution: | → fixed |
Status: | new → closed |
comment:18 Changed 6 years ago by
Hi, There is another ticket requesting to back-port this feature to 1.4 (or even 1.3). Closed that one as duplicate #20087.
This is a minor feature and can safely be dealt with post-1.0.