From eeb9937544ac7813bcbe387635cee09e277ccf8a Mon Sep 17 00:00:00 2001 From: Francesco Giacomini <francesco.giacomini@cnaf.infn.it> Date: Thu, 11 Nov 2021 15:44:44 +0000 Subject: [PATCH] Adapt test to openresty 1.19.9.1 Apparently the bug mentioned in the (removed) comment has been fixed. But this showed a bug in our test: http vs HTTP in the pattern to match in the log. Issue #17 --- t/ssl_log_voms_plain_http.t | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/t/ssl_log_voms_plain_http.t b/t/ssl_log_voms_plain_http.t index 1989c41..00d25ec 100644 --- a/t/ssl_log_voms_plain_http.t +++ b/t/ssl_log_voms_plain_http.t @@ -1,10 +1,4 @@ -# This test is always successful because, for some reason (a bug?) -# the error.log as seen in Test::Nginx doesn't contain the entries for -# the master process, although they are evailable in the actual file. -# As a consequence the no_error_log check is always satisfied, -# even if the segmentation fault were present - use Test::Nginx::Socket 'no_plan'; master_on(); @@ -55,7 +49,5 @@ GET /lua --- error_log client prematurely closed connection retrieve_voms_ac_from_proxy -plain http ---- no_error_log -signal 11 +plain HTTP --- error_code: 200 -- GitLab