diff --git a/exodus_lambda/functions/origin_request.py b/exodus_lambda/functions/origin_request.py index 9ef48948..9ca95471 100755 --- 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 index e45f3794..911c832e 100644 --- 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,