Message236863
| Author |
demian.brecht |
| Recipients |
demian.brecht, jimr, martin.panter |
| Date |
2015年02月28日.01:24:42 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1425086684.07.0.173769602199.issue23539@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
> Also, I think OPTIONS should be removed from the list of methods that enforce a Content-Length. I wouldn’t normally expect any payload for OPTIONS, since RFC 7231 explicitly says it does not define a use for a payload, but requires a Content-Type if a payload is sent.
I think Martin's right about this and it's consistent with my concern about servers treating the existence of a Content-Length header as an indicator of a request body.
In any event, with the new logic, a Content-Length header will be added if the body is not None.
> I've just put the logic inside _set_content_length
Fair enough. It /does/ encapsulate all logic around setting the content length within the method.
I've left a couple minor comments in Rietveld. In addition to what's there, docs should also be updated to reflect the new behaviour.
Thanks for the work on this! |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2015年02月28日 01:24:44 | demian.brecht | set | recipients:
+ demian.brecht, martin.panter, jimr |
| 2015年02月28日 01:24:44 | demian.brecht | set | messageid: <1425086684.07.0.173769602199.issue23539@psf.upfronthosting.co.za> |
| 2015年02月28日 01:24:43 | demian.brecht | link | issue23539 messages |
| 2015年02月28日 01:24:42 | demian.brecht | create |
|