From 57153079a53b1738436e55a8b07f0e57b6e43615 Mon Sep 17 00:00:00 2001 From: Trustin Lee Date: Mon, 14 Jan 2013 22:37:05 +0900 Subject: [PATCH] Chrome requires the :version: header in SPDY_SESSION_PUSHED_SYN_STREAM frames otherwise it issues a HTTP_TRANSACTION_SPDY_SEND_REQUEST_HEADERS and a SPDY_STREAM_ERROR saying "HEADERS incomplete headers, but pending data frames." Ported from 3 --- .../main/java/io/netty/handler/codec/spdy/SpdyHttpEncoder.java | 1 + 1 file changed, 1 insertion(+) diff --git a/codec-http/src/main/java/io/netty/handler/codec/spdy/SpdyHttpEncoder.java b/codec-http/src/main/java/io/netty/handler/codec/spdy/SpdyHttpEncoder.java index cdb15c34fb..6a6ae82cf6 100644 --- a/codec-http/src/main/java/io/netty/handler/codec/spdy/SpdyHttpEncoder.java +++ b/codec-http/src/main/java/io/netty/handler/codec/spdy/SpdyHttpEncoder.java @@ -249,6 +249,7 @@ public class SpdyHttpEncoder extends MessageToMessageEncoder { HttpResponse httpResponse = (HttpResponse) httpMessage; SpdyHeaders.setStatus(spdyVersion, spdySynStreamFrame, httpResponse.getStatus()); SpdyHeaders.setUrl(spdyVersion, spdySynStreamFrame, URL); + SpdyHeaders.setVersion(spdyVersion, spdySynStreamFrame, httpMessage.getProtocolVersion()); spdySynStreamFrame.setUnidirectional(true); }