http2: Fix bug that associated stream canceled on PUSH_PROMISE

Previously we don't ignore PUSH_PROMISE header fields in on_header
callback.  It makes header values mixed with following HEADERS,
resulting protocol error.

diff --git a/lib/http2.c b/lib/http2.c
index dac9ab4a106f5abc8ffddb81181b2be7ad811b0f..31949413bb229b1d9bfce441efc2f937ab68e4a2 100644 (file)
--- a/lib/http2.c
+++ b/lib/http2.c
@@ -424,6 +424,11 @@ static int on_header(nghttp2_session *session, const nghttp2_frame *frame,
   (void)frame;
   (void)flags;
 
+  /* Ignore PUSH_PROMISE for now */
+  if(frame->hd.type != NGHTTP2_HEADERS) {
+    return 0;
+  }
+
   if(frame->hd.stream_id != c->stream_id) {
     return 0;
   }