Ticket #6080 enhancement closed fixed

Opened 8 months ago

Last modified 8 months ago

Port twisted.web.http_headers to Python 3

Reported by: exarkun Owned by: exarkun
Priority: normal Milestone: Python 3.3 Minimal
Component: web Keywords:
Cc: jknight Branch: branches/http-headers-py3-6080
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

This is a dependency of both the client and the server.

Change History

1

Changed 8 months ago by DefaultCC Plugin

  • cc jknight added

2

Changed 8 months ago by exarkun

  • branch set to branches/http-headers-py3-6080
  • branch_author set to exarkun

(In [36003]) Branching to 'http-headers-py3-6080'

3

Changed 8 months ago by exarkun

  • owner changed from exarkun to itamar
  • keywords review added

Switched it to MutableMapping which wasn't too bad, but it's not quite mechanical.  Build results.

4

Changed 8 months ago by thijs

#5829 was opened for MutableMapping, so that is either a duplicate or it should be referenced/closed when merging this ticket.

5

Changed 8 months ago by itamar

  • owner changed from itamar to exarkun
  • keywords review removed

Looks good, with two minor fixes; merge when done:

  1. Add twisted.web.test to _twistedpython3.py.
  2. Some of the methods still refer to C{str} and strings instead of bytes, e.g. hasHeader, removeHeader, etc..

6

Changed 8 months ago by exarkun

(In [36040]) Address review comments

refs #6080

7

Changed 8 months ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(In [36042]) Merge http-headers-py3-6080

Author: exarkun Reviewer: itamarst Fixes: #6080

Port twisted.web.http_headers to Python 3. Also fix some structural documentation issues.

Note: See TracTickets for help on using tickets.