From fafd1b0a5b3b6ee0c7d7dd288629c992fc63ef10 Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Tue, 3 Nov 2020 18:25:52 +0100 Subject: [PATCH] CLEANUP: mux-h2: Remove the h1 parser state from the h2 stream Since the h2 multiplexer no longer relies on the legacy HTTP representation, and uses exclusively the HTX, the H1 parser state (h1m) is no longer used by the h2 streams. Thus it can be removed. This patch may be backported as far as 2.1. --- src/mux_h2.c | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/src/mux_h2.c b/src/mux_h2.c index aa0a76029..740e25555 100644 --- a/src/mux_h2.c +++ b/src/mux_h2.c @@ -14,7 +14,6 @@ #include #include #include -#include #include #include #include @@ -192,13 +191,12 @@ enum h2_ss { /* H2 stream descriptor, describing the stream as it appears in the H2C, and as - * it is being processed in the internal HTTP representation (H1 for now). + * it is being processed in the internal HTTP representation (HTX). */ struct h2s { struct conn_stream *cs; struct session *sess; struct h2c *h2c; - struct h1m h1m; /* request or response parser state for H1 */ struct eb32_node by_id; /* place in h2c's streams_by_id */ int32_t id; /* stream ID */ uint32_t flags; /* H2_SF_* */ @@ -1438,16 +1436,6 @@ static struct h2s *h2s_new(struct h2c *h2c, int id) h2s->body_len = 0; h2s->rxbuf = BUF_NULL; - if (h2c->flags & H2_CF_IS_BACK) { - h1m_init_req(&h2s->h1m); - h2s->h1m.err_pos = -1; // don't care about errors on the request path - h2s->h1m.flags |= H1_MF_TOLOWER; - } else { - h1m_init_res(&h2s->h1m); - h2s->h1m.err_pos = -1; // don't care about errors on the response path - h2s->h1m.flags |= H1_MF_TOLOWER; - } - h2s->by_id.key = h2s->id = id; if (id > 0) h2c->max_id = id;