From: Rainer Jung Date: Sun, 19 Jan 2025 19:32:14 +0000 (+0000) Subject: *) mod_http2: fix test_h2_200_17+18 for newer curl versions. Just add log X-Git-Tag: 2.4.63-candidate~10 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0acf2161c6dd45020d794cc482f46b5d93883ce9;p=thirdparty%2Fapache%2Fhttpd.git *) mod_http2: fix test_h2_200_17+18 for newer curl versions. Just add log output and add to test checks. No functional change. Merges r1923187 from trunk. git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1923238 13f79535-47bb-0310-9956-ffa450edef68 --- diff --git a/STATUS b/STATUS index 0efa26b2c6a..680787005bd 100644 --- a/STATUS +++ b/STATUS @@ -157,10 +157,6 @@ RELEASE SHOWSTOPPERS: PATCHES ACCEPTED TO BACKPORT FROM TRUNK: [ start all new proposals below, under PATCHES PROPOSED. ] - *) mod_http2: fix test_h2_200_17+18 for newer curl versions. Just add log - output and add to test checks. No functional change. - 2.4.x patch: svn merge -c 1923187 ^/httpd/httpd/trunk . - +1: icing, jim, rjung PATCHES PROPOSED TO BACKPORT FROM TRUNK: [ New proposals should be added at the end of the list ] diff --git a/modules/http2/h2_session.c b/modules/http2/h2_session.c index 7c73e230699..7fa0376043a 100644 --- a/modules/http2/h2_session.c +++ b/modules/http2/h2_session.c @@ -363,9 +363,11 @@ static int on_frame_recv_cb(nghttp2_session *ng2s, else { ap_log_cerror(APLOG_MARK, APLOG_DEBUG, 0, session->c1, H2_SSSN_LOG(APLOGNO(03066), session, - "recv FRAME[%s], frames=%ld/%ld (r/s)"), + "recv FRAME[%s], frames=%ld/%ld (r/s), " + "remote.emitted=%d"), buffer, (long)session->frames_received, - (long)session->frames_sent); + (long)session->frames_sent, + (int)session->remote.emitted_count); } } diff --git a/test/modules/http2/test_200_header_invalid.py b/test/modules/http2/test_200_header_invalid.py index cbc4b6c9fa8..6b73301c282 100644 --- a/test/modules/http2/test_200_header_invalid.py +++ b/test/modules/http2/test_200_header_invalid.py @@ -243,7 +243,8 @@ class TestInvalidHeaders: conf.add_vhost_cgi() conf.install() assert env.apache_restart() == 0 - re_emitted = re.compile(r'.* AH03401: .* shutdown, remote.emitted=1') + re_emitted = re.compile(r'.* (AH03401: .* shutdown,|' + r'AH03066: .* FRAME\[GOAWAY.*) remote.emitted=1') url = env.mkurl("https", "cgi", "/") opt = [] for i in range(10): @@ -263,7 +264,8 @@ class TestInvalidHeaders: conf.add_vhost_cgi() conf.install() assert env.apache_restart() == 0 - re_emitted = re.compile(r'.* AH03401: .* shutdown, remote.emitted=1') + re_emitted = re.compile(r'.* (AH03401: .* shutdown,|' + r'AH03066: .* FRAME\[GOAWAY.*) remote.emitted=1') url = env.mkurl("https", "cgi", "/") opt = [] for i in range(100):