Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

express-spdy server sends a connection:keep-alive header #1

Open
gdewan opened this issue Aug 4, 2011 · 1 comment
Open

express-spdy server sends a connection:keep-alive header #1

gdewan opened this issue Aug 4, 2011 · 1 comment

Comments

@gdewan
Copy link

gdewan commented Aug 4, 2011

The express-spdy server sends a connection:keep-alive header in the syn_reply frame, even though the original syn_stream did not have a connection header. As far as I know, connection headers are useless in SPDY.

t=1312475149318 [st= 0] SPDY_SESSION_SYN_STREAM
--> flags = 1
--> accept: text/html,application/xhtml+xml,application/xml;q=0.9,/;q=0.8
accept-charset: ISO-8859-1,utf-8;q=0.7,*;q=0.3
accept-encoding: gzip,deflate,sdch
accept-language: en-US,en;q=0.8
host: sfo-lpwng:3000
method: GET
scheme: https
url: /
user-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_6_7) AppleWebKit/535.1 (KHTML, like Gecko) Chrome/13.0.782.107 Safari/535.1
version: HTTP/1.1
--> id = 1
t=1312475149323 [st= 5] SPDY_SESSION_SYN_REPLY
--> flags = 0
--> connection: keep-alive
content-length: 689
content-type: text/html
status: 200 OK
version: HTTP/1.1
x-powered-by: Express
--> id = 1

@eee-c
Copy link
Owner

eee-c commented Aug 5, 2011

Thanks! That's definitely a bug. Connection headers are forbidden in SPDY, so I'll have to suppress that...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants