From cc94ffa0b1ef8326bd7ef2a4c5e38ee6f976d568 Mon Sep 17 00:00:00 2001 From: Bertrand Demiddelaer Date: Mon, 13 Sep 2021 12:06:04 +0200 Subject: [PATCH] RELENG-4905 allow copy operation without Script-Name header --- conf/nginx.conf.template | 6 ------ tests/end2end/test_simple.py | 2 +- 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/conf/nginx.conf.template b/conf/nginx.conf.template index 2f975ef..95d0e35 100644 --- a/conf/nginx.conf.template +++ b/conf/nginx.conf.template @@ -567,12 +567,6 @@ http { return 400; } - # Not allowed behind ingress. - # - if ($http_script_name) { - return 403; - } - # Canonicalize key. # set_by_lua_file $canonical_path /etc/nginx/canonicalize_path.ljbc; diff --git a/tests/end2end/test_simple.py b/tests/end2end/test_simple.py index a88cae9..f2dd02c 100644 --- a/tests/end2end/test_simple.py +++ b/tests/end2end/test_simple.py @@ -143,7 +143,7 @@ def test_simple_upload_and_copy_behind_ingress(self): ) success = 'SUCCESSFUL'.encode('utf-8') copy = self.session.get(url, data=success, headers={'Script-Name': '/foo'}) - assert copy.status_code == 403 + assert copy.status_code == 200 # Upload without ingress and download with ingress url = '{artifacts_url}/upload/{container}/.final_status'.format(