Merge pull request #514 from negillett/18071

Increase request querystring limit [RHELDST-18071]
release-engineering · Oct 18, 2023 · 844ce9c · 844ce9c
2 parents 56ccd7a + b60a7fb
commit 844ce9c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/exodus_lambda/functions/origin_request.py b/exodus_lambda/functions/origin_request.py
@@ -335,7 +335,7 @@ def validate_request(self, request):
         if not 0 < len(request["uri"]) < 2000:
             self.logger.error("uri exceeds length limits: %s", request["uri"])
             valid = False
-        if "querystring" in request and not len(request["querystring"]) < 2000:
+        if "querystring" in request and not len(request["querystring"]) < 3000:
             self.logger.error(
                 "querystring exceeds length limits: %s", request["querystring"]
             )

diff --git a/tests/functions/test_origin_request.py b/tests/functions/test_origin_request.py
@@ -263,7 +263,7 @@ def test_origin_request_fail_querystring_validation(caplog):
     # Validation fails for too lengthy URIs.
     event = {
         "Records": [
-            {"cf": {"request": {"uri": "/", "querystring": "o" * 2001}}}
+            {"cf": {"request": {"uri": "/", "querystring": "o" * 3001}}}
         ]
     }
 
@@ -289,7 +289,7 @@ def test_origin_request_fail_querystring_validation(caplog):
             "level": "ERROR",
             "time": mock.ANY,
             "aws-request-id": None,
-            "message": "querystring exceeds length limits: %s" % ("o" * 2001),
+            "message": "querystring exceeds length limits: %s" % ("o" * 3001),
             "logger": "origin-request",
             "request": None,
             "response": None,