#8121 closed (fixed)
LocaleMiddleware should not set Content-Language header if already present
| Reported by: | Joost Cassee | Owned by: | nobody |
|---|---|---|---|
| Component: | Internationalization | Version: | dev |
| Severity: | Keywords: | ||
| Cc: | 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
The user may have added the Content-Language header to the Response objects explicitely. In that case LocaleMiddleware should not overwrite it. Attached patch fixes this.
Attachments (1)
Change History (4)
by , 17 years ago
| Attachment: | 8121-r8137.diff added |
|---|
comment:1 by , 17 years ago
| milestone: | → 1.0 |
|---|---|
| Triage Stage: | Unreviewed → Ready for checkin |
comment:2 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
(In [8259]) Fixed #8121 -- Don't override the Content-Language HTTP header in the locale
middleware if it's already been set. Thanks, jcassee.