molnarg / node-http2

An HTTP/2 client and server implementation for node.js
MIT License
1.79k stars 185 forks source link

Avoid firing window_update unnecessarily. #171

Closed tbetbetbe closed 8 years ago

tbetbetbe commented 8 years ago

Currently, the following scenario can occur

This change resolves the problem by only firing the 'window_update' event if the window changes. This is a simple way of prevent the SETTINGS receipt window_update events taking up window_update events registered by reads

nwgh commented 8 years ago

Thanks!