twitter-archive/CocoaSPDY

Invalid SETTINGS frame sent

indutny opened this issue · 3 comments

indutny commented

While trying to figure out: spdy-http2/node-spdy#133

I have found that CocoaSPDY sends SETTINGS_INITIAL_WINDOW_SIZE set to 0, is there any particular reasons for this? Or is it just a bug?

indutny commented

Quote from specification:

When the value of SETTINGS_INITIAL_WINDOW_SIZE changes, a receiver MUST adjust the size of all stream flow control windows that it maintains by the difference between the new value and the old value.

Also this probably means that servers that are working with CocoaSPDY are broken and ignoring specification.

goaway commented

Sorry didn't mean to close this - thanks for reporting the issue, I'll be looking into it shortly.

goaway commented

Hey @indutny, thanks for your help in looking into this! It looks like this was caused by a project that was using a SPDYConfiguration object that had all values set to 0. This is confusing and I'll consider changing the result of [[SPDYConfiguration alloc] init] in a future release. The current correct way to utilize default values is to simply not set your own custom configuration, or to base your configuration off the return value of [SPDYConfiguration defaultConfiguration].