W3C home > Mailing lists > Public > ietf-http-wg@w3.org > January to March 2015

Re: IETF LC comments on draft-ietf-httpbis-header-compression-10

From: Matthew Kerwin <matthew@kerwin.net.au>
Date: Sun, 11 Jan 2015 09:16:55 +1000
Message-ID: <CACweHNAPwoCggR9jnCLiaU+daXY-vKGh5PGrTP=0URNaWwgaxg@mail.gmail.com>
To: Julian Reschke <julian.reschke@greenbytes.de>
Cc: "ietf@ietf.org" <ietf@ietf.org>, "ietf-http-wg@w3.org" <ietf-http-wg@w3.org>
On 11 January 2015 at 02:58, Julian Reschke <julian.reschke@greenbytes.de>
wrote:

>
> 5.1.  Integer Representation
>
>
> We have bad experience with making indentation in artwork significant;
> maybe introducing brackets would be a good idea.
> ​
> ​
>
> ​
> ​


(And all the Python advocates cry)


​
>
> ​​
>    This integer representation allows for values of indefinite size.  It
>    is also possible for an encoder to send a large number of zero
>    values, which can waste octets and could be used to overflow integer
>    values.  Excessively large integer encodings - in value or octet
>    length - MUST be treated as a decoding error.  Different limits can
>    be set for each of the different uses of integers, based on
>    implementation constraints.
>
> Having a MUST here when we don't say what "excessive" is seems like a bad
> idea.
>
>
This seems like a MAY -- explaining the error you receive, rather than
telling you when to raise one.



> 5.2.  String Literal Representation
>
> The figure could be interpreted as if the string length is always
> expressed in 7 bits, which I believe is not true.
>
> 6.2.1.  Literal Header Field with Incremental Indexing
>
>
> See above. One can read this as "H" being bit 0 in the second octet; but
> that's not always the case. I think it would be good to revise the figure
> format.
>
>
It could help to add a line of just:   | ... |  below any (N+) element.
Maybe.



-- 
  Matthew Kerwin
  http://matthew.kerwin.net.au/
Received on Saturday, 10 January 2015 23:17:22 UTC

This archive was generated by hypermail 2.3.1 : Tuesday, 1 March 2016 11:11:36 UTC