Opened 4 years ago

Closed 3 years ago

#4911 defect closed fixed (fixed)

twisted.web.http docstring indicates chunked transfer is not supported

Reported by: andy Owned by: exarkun
Priority: normal Milestone:
Component: web Keywords:
Cc: jknight Branch:
Author: Launchpad Bug:

Description

From the module docstring list of future plans:

" - Accept chunked data from clients in server."

whereas later in the file there's a bunch of support for it, and it appears to work fine.

Attachments (1)

http-docstring.patch (781 bytes) - added by exarkun 4 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 4 years ago by DefaultCC Plugin

  • Cc jknight added

Changed 4 years ago by exarkun

comment:2 Changed 4 years ago by exarkun

  • Keywords review added

comment:3 Changed 3 years ago by itamar

  • Keywords review removed
  • Owner set to exarkun
  1. Needs .misc news file.
  1. The docstring isn't quite a accurate - some of the code is used by _newclient, e.g. the chunked encoding parser. So maybe mention "Plus some utility classes used by the HTTP client" (PotentialDataLoss is the only public thing a client *user* would end up knowing about, I think.)
  1. It'd be nice if my name was removed as maintainer. Not essential though.

Other than that, looks good. Feel free to commit with #1, and #2/#3 if you feel like it.

comment:4 Changed 3 years ago by exarkun

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

(In [30897]) Update the twisted.web.http module docstring to more accurately reflect that module's current state

Author: exarkun
Reviewer: itamar
Fixes: #4911

Note: See TracTickets for help on using tickets.