aboutsummaryrefslogtreecommitdiff
path: root/tests/data
diff options
context:
space:
mode:
authorTatsuhiro Tsujikawa <tatsuhiro.t@gmail.com>2015-04-02 00:28:58 +0900
committerDaniel Stenberg <daniel@haxx.se>2015-04-02 15:19:29 +0200
commit21e82bd635124881a4a2ac08b3b9b28efe98afee (patch)
treeb619753f1d34ba0addcaeb9d63df46c464c4dda4 /tests/data
parent2685041a5c285968696de639699fb39a0fdf8c69 (diff)
http2: Fix missing nghttp2_session_send call in Curl_http2_switched
Previously in Curl_http2_switched, we called nghttp2_session_mem_recv to parse incoming data which were already received while curl was handling upgrade. But we didn't call nghttp2_session_send, and it led to make curl not send any response to the received frames. Most likely, we received SETTINGS from server at this point, so we missed opportunity to send SETTINGS + ACK. This commit adds missing nghttp2_session_send call in Curl_http2_switched to fix this issue. Bug: https://github.com/bagder/curl/issues/192 Reported-by: Stefan Eissing
Diffstat (limited to 'tests/data')
-rw-r--r--tests/data/test18015
1 files changed, 2 insertions, 3 deletions
diff --git a/tests/data/test1801 b/tests/data/test1801
index a95e76a22..0aedfed51 100644
--- a/tests/data/test1801
+++ b/tests/data/test1801
@@ -23,7 +23,6 @@ lies!
<datacheck>
HTTP/1.1 101 Switching!
-
</datacheck>
# listen to the upgrade request!
@@ -62,9 +61,9 @@ Upgrade: %H2CVER
HTTP2-Settings: AAMAAABkAAQAAP__
</protocol>
-# nothing is returned, because no HTTP/2 data nor headers was handled
+# CURLE_HTTP2: Send failure: Broken pipe
<errorcode>
-56
+16
</errorcode>
</verify>
</testcase>