spdy-http2 / node-spdy

SPDY server on Node.js
2.81k stars 196 forks source link

Priotirization support #296

Closed VojtaStanek closed 7 years ago

VojtaStanek commented 7 years ago

It seems that there is no prioritisation support.

It should respect browser-defined stream priority (as defined in section 5.3 of HTTP/2 specs) as well as support specifying priority when pushing resources.

(I haven't found any indication of implementation of this.)

See: http://httpwg.org/specs/rfc7540.html#StreamPriority https://nghttp2.org/blog/2014/04/27/how-dependency-based-prioritization-works/

indutny commented 7 years ago

It actually works, but on the level of spdy-transport: https://github.com/indutny/spdy-transport/blob/master/lib/spdy-transport/priority.js .

VojtaStanek commented 7 years ago

Am I able / how to set pushed resources weight in dependency tree? And set other requests' dependency tree?

On Mon, 28 Nov 2016 1:38 am Fedor Indutny, notifications@github.com wrote:

It actually works, but on the level of spdy-transport: https://github.com/indutny/spdy-transport/blob/master/lib/spdy-transport/priority.js .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/indutny/node-spdy/issues/296#issuecomment-263160004, or mute the thread https://github.com/notifications/unsubscribe-auth/AEFagn5rGwDwRMPfK6UzUZJdvOpVvDo0ks5rCiJqgaJpZM4K9SIm .

worenga commented 7 years ago

Am I able / how to set pushed resources weight in dependency tree? And set other requests' dependency tree?

I would really be interested in that as well. It seems pushed objects are interleaved with the main response. Any idea how to avoid this (on the server side)?