kaazing / http2-cache.js

2 stars 11 forks source link

implement maximumReconnectInterval and accelerationStrategy support on Configuration and Xhr #102

Closed hthetiot closed 6 years ago

hthetiot commented 6 years ago

See #101 for details spec

Main logic entry point:

hthetiot commented 6 years ago

Need to check config.proxy exist before forEach

hthetiot commented 6 years ago

Via @dpwspoon

When in the accelerationStrategy: connected it looks like any initial requests that beat the establishing the initial ws connection/attempt will bypass the cache. (correct me if I’m mistaken).

This will add more load to the origin server then I was expecting as all browsers in that mode on initial startup might make requests directly to origin even though they will switch back to being accelerated shortly.

Perhaps we should wait for the first attempt to succeed or fail before falling back OR add a new acceleration strategy that does that

dpwspoon commented 6 years ago

Looks ok to me