search

tatsuhiro-t / spdylay

The experimental SPDY protocol version 2, 3 and 3.1 implementation in C
http://tatsuhiro-t.github.io/spdylay/
MIT License
603 stars 102 forks source link

about spdy/2 priority #98

Closed chobits closed 10 years ago

chobits commented 10 years ago

Hi tatsuhiro, In spdylay, I see that:

define SPDYLAY_PRI_LOWEST_SPDY2 3

In google spdy/2 draft [1]: 0 represents the lowest priority and 3 represents the highest priority.

In google spdy/3 draft [2]:

  1. Incompatibilities with SPDY draft # 2 Fixed bug: Priorities were described backward (0 was lowest instead of highest).

Is this a minor bug or am I missing something?

[1] http://www.chromium.org/spdy/spdy-protocol/spdy-protocol-draft2 [2] http://www.chromium.org/spdy/spdy-protocol/spdy-protocol-draft3#TOC-7.-Incompatibilities-with-SPDY-draft-2

tatsuhiro-t commented 10 years ago

I completely missed it. But it seems that chrome also uses 0 as highest priority even in spdy/2. Since spdy/2 is going to disappear slowly, I think this is just a spec bug and WONTFIX.

chobits commented 10 years ago

Thanks for reply. It is spdy/2 draft bug. Firefox, chrome, apache mod_spdy and nginx have fixed this, they uses 0 as highest priority for spdy/2.