Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-16 Thread William A Rowe Jr
On Jun 15, 2015 6:11 PM, Roy T. Fielding field...@gbiv.com wrote: On Jun 15, 2015, at 9:33 AM, William A Rowe Jr wr...@rowe-clan.net wrote: Reviewing the spec, I cannot find where Sambar server is permitted to insert whitespace. I further reviewed the ABNF appendix, and it does not appear

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-16 Thread Jim Jagielski
Sooo in order to get 2.4.15 out, it would be nice to have a patch :)

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-16 Thread Yann Ylavic
On Tue, Jun 16, 2015 at 7:22 PM, Jim Jagielski j...@jagunet.com wrote: Sooo in order to get 2.4.15 out, it would be nice to have a patch :) Isn't the one proposed in STATUS suitable (section SHOWSTOPPERS)? It has been positively tested by Steffen in [1] and also passes the new framework

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-16 Thread William A Rowe Jr
Note in STATUS I've requested that you split the approved patch from security@ that seemed to be lost in long and winding patch versioning from the spaces accepted. A patch should correct one thing, not several, it makes these too difficult to review when folks have a small window of free time.

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-16 Thread Yann Ylavic
On Tue, Jun 16, 2015 at 10:50 PM, Yann Ylavic ylavic@gmail.com wrote: On Tue, Jun 16, 2015 at 8:09 PM, William A Rowe Jr wr...@rowe-clan.net wrote: Note in STATUS I've requested that you split the approved patch from security@ that seemed to be lost in long and winding patch versioning

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-16 Thread Yann Ylavic
On Tue, Jun 16, 2015 at 8:09 PM, William A Rowe Jr wr...@rowe-clan.net wrote: Note in STATUS I've requested that you split the approved patch from security@ that seemed to be lost in long and winding patch versioning from the spaces accepted. A patch should correct one thing, not several, it

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-15 Thread Graham Leggett
On 15 Jun 2015, at 7:00 PM, Jeff Trawick traw...@gmail.com wrote: 1.3 (or 1.3-based servers) put whitespace there. 1.3.x, 2.0.x, 2.2.x, and 2.4.x (for all released x so far) accepts whitespace there. We can't change that by default in a stable branch. +1. We need to be liberal in what we

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-15 Thread William A Rowe Jr
On Jun 15, 2015 1:26 PM, Graham Leggett minf...@sharp.fm wrote: On 15 Jun 2015, at 7:00 PM, Jeff Trawick traw...@gmail.com wrote: 1.3 (or 1.3-based servers) put whitespace there. 1.3.x, 2.0.x, 2.2.x, and 2.4.x (for all released x so far) accepts whitespace there. We can't change that by

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-15 Thread Ruediger Pluem
On 06/15/2015 07:00 PM, Jeff Trawick wrote: On Mon, Jun 15, 2015 at 12:33 PM, William A Rowe Jr wr...@rowe-clan.net mailto:wr...@rowe-clan.net wrote: 1.3 (or 1.3-based servers) put whitespace there. 1.3.x, 2.0.x, 2.2.x, and 2.4.x (for all released x so far) accepts whitespace there.

TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-15 Thread William A Rowe Jr
Reviewing the spec, I cannot find where Sambar server is permitted to insert whitespace. I further reviewed the ABNF appendix, and it does not appear there, either. The spec seems unambiguous; chunk = chunk-size [ chunk-ext ] CRLF chunk-data CRLF chunk-size =

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-15 Thread Jeff Trawick
On Mon, Jun 15, 2015 at 12:33 PM, William A Rowe Jr wr...@rowe-clan.net wrote: Reviewing the spec, I cannot find where Sambar server is permitted to insert whitespace. I further reviewed the ABNF appendix, and it does not appear there, either. The spec seems unambiguous; chunk =

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-15 Thread Jim Jagielski
1.3 (or 1.3-based servers) put whitespace there. 1.3.x, 2.0.x, 2.2.x, and 2.4.x (for all released x so far) accepts whitespace there. We can't change that by default in a stable branch. ++1

Re: TWS ; LWS permitted by RFC 7230 4.1.1? Apparently, no.

2015-06-15 Thread Roy T. Fielding
On Jun 15, 2015, at 9:33 AM, William A Rowe Jr wr...@rowe-clan.net wrote: Reviewing the spec, I cannot find where Sambar server is permitted to insert whitespace. I further reviewed the ABNF appendix, and it does not appear there, either. Right, this was a deliberate decision to reduce