Opened 3 years ago

Closed 3 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
(diff, github, buildbot, log)
Author: therve Launchpad Bug:

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 3 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 3 years ago by DefaultCC Plugin

  • Cc jknight added

comment:2 Changed 3 years ago by ivank

  • Summary changed from twisted.web.http_headers.Headers capitalize some headers incorrectly to twisted.web.http_headers.Headers capitalizes some headers incorrectly

comment:3 Changed 3 years ago by ivank

  • Keywords review easy added

comment:4 Changed 3 years ago by therve

  • Owner set to therve

comment:5 Changed 3 years ago by therve

  • Author set to therve
  • Branch set to branches/capitalize-headers-5054

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

comment:6 Changed 3 years ago by therve

(In [31707]) Apply patch

Refs #5054

comment:7 Changed 3 years ago by therve

  • Keywords review removed

Looks good!

comment:8 Changed 3 years ago by therve

  • Resolution set to fixed
  • Status changed from new to closed

(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.