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

Re: Stream State and PRIORITY Frames

From: laike9m <laike9m@gmail.com>
Date: Fri, 20 Jan 2017 21:17:02 +0800
Message-ID: <CAJutW=dX1Z8z+XJ10+uX729bxgWh417kXdd27kg=RO6VbcbvgA@mail.gmail.com>
To: Scott Mitchell <scott.k.mitch1@gmail.com>
Cc: Martin Thomson <martin.thomson@gmail.com>, Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com>, HTTP Working Group <ietf-http-wg@w3.org>
If you don’t include RESERVED streams in the count for
SETTINGS_MAX_CONCURRENT_STREAMS then how do you limit the amount of
RESERVED streams, and how does your peer know about this limit? I have
imposed an implementation specific metric in the past, but this seems less
preferable than relying on something in the RFC that the peer is aware of.
Either way having infinite of something doesn’t work in practice.

As Martin has explained, H2 doesn’t limit the amount of RESERVED streams,
based on the notion that HEADERS are close to free. It’s true that one
trying to send infinite number of PUSH_PROMISEs to client can cause
problems, but 1. This only happens if the server is malicious, and if it’s
malicious, having a limit in the RFC won’t prevent anything, and 2. Not
counting PUSH_PROMISEs is a tradeoff for fast delivery of PUSH_PROMISEs,
which stops client from sending more requests. I guess this is what Martin
meant by “If you limit server push by applying a stream limit, then you
prevent it from being used in time for the client to use it.”

(Forgot to reply to all :P)

On Thu, Jan 19, 2017 at 9:21 AM, Scott Mitchell <scott.k.mitch1@gmail.com>
wrote:

>
>
> On Tue, Jan 17, 2017 at 2:43 PM, Scott Mitchell <scott.k.mitch1@gmail.com>
> wrote:
>
>> From my perspective I would like to see two clarifications:
>>
>> 1. It is clear to me that PRIORITY doesn't impact state.
>>
>
> Just to clarify ... it is clear that a PRIORITY frame doesn't impact the
> state of the stream  it is carrying priority information for. The impacts
> PRIORITY frames have on other streams is not clear due to the wording in
> section 5.1.1.
>
>
>> However Section 5.1.1 states "first use of a new stream identifier"
>> which makes no reference to stream state. If stream state is
>> important/implied here better to be specific about it. I don't think the
>> one-off example below this text is sufficient to convey the intended
>> implications of this statement.
>>
>> 2. Section 5.1.2 states "Streams in either of the 'reserved' states do
>> not count toward the stream limit." which seems to conflict with section
>> 8.2.2 "A client can use the SETTINGS_MAX_CONCURRENT_STREAMS setting to
>> limit the number of responses that can be concurrently pushed by a
>> server.". These two statements appear to contradict each other. Since
>> SETTINGS_MAX_CONCURRENT_STREAMS is really the only mechanism to limit
>> resources due to server push I'm assuming section 5.1.2 is overly
>> restrictive.
>>
>>
>> On Tue, Jan 17, 2017 at 2:27 PM, Martin Thomson <martin.thomson@gmail.com
>> > wrote:
>>
>>> On 18 January 2017 at 01:37, Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com>
>>> wrote:
>>> > If my understanding is correct, this only refers to the new stream ID
>>> used
>>> > by HEADERS, and PUSH_PROMISE frames which open or reserve streams.  The
>>> > example text following that statement uses HEADERS which opens new
>>> stream.
>>> > PRIORITY frame does not change stream state, and there is no reason to
>>> close
>>> > all unused streams lower than bearing stream ID.  That said, I agree
>>> that
>>> > this is not crystal clear in the document.  In practice, this is
>>> probably
>>> > rather rare case.
>>>
>>> This is, I think, the expectation.
>>>
>>> I think that we probably want to clarify the point by explicitly
>>> saying that PRIORITY doesn't affect stream states.  We say that it can
>>> be sent in any state, but we don't also mention that important point.
>>> Do people here agree that an erratum on this point is appropriate
>>> here?
>>>
>>
>>
>
Received on Friday, 20 January 2017 13:17:35 UTC

This archive was generated by hypermail 2.3.1 : Friday, 20 January 2017 13:17:37 UTC