Opened 6 years ago

Closed 6 years ago

#5054 defect closed fixed (fixed)

twisted.web.http_headers.Headers capitalizes some headers incorrectly

Reported by: ivank Owned by: therve
Priority: normal Milestone:
Component: web Keywords: easy
Cc: ivank, jknight Branch: branches/capitalize-headers-5054
branch-diff, diff-cov, branch-cov, buildbot
Author: therve

Description

There are a few headers besides WWW-Authenticate that need to be in _caseMappings.

(Patch incoming. This should be pretty simple.)

Attachments (1)

5054-00-capitalize-headers-correctly.patch (2.3 KB) - added by ivank 6 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 6 years ago by DefaultCC Plugin

Cc: jknight added

comment:2 Changed 6 years ago by ivank

Summary: twisted.web.http_headers.Headers capitalize some headers incorrectlytwisted.web.http_headers.Headers capitalizes some headers incorrectly

Changed 6 years ago by ivank

comment:3 Changed 6 years ago by ivank

Keywords: review easy added

comment:4 Changed 6 years ago by therve

Owner: set to therve

comment:5 Changed 6 years ago by therve

Author: therve
Branch: branches/capitalize-headers-5054

(In [31706]) Branching to 'capitalize-headers-5054'

comment:6 Changed 6 years ago by therve

(In [31707]) Apply patch

Refs #5054

comment:7 Changed 6 years ago by therve

Keywords: review removed

Looks good!

comment:8 Changed 6 years ago by therve

Resolution: fixed
Status: newclosed

(In [31708]) Merge capitalize-headers-5054

Author: ivank Reviewer: therve Fixes: #5054

Correctly capitalize specific headers in twisted.web.http_headers.Headers.

Note: See TracTickets for help on using tickets.