fix(apigw): add all safe URI chars RFC3986 pt2

heitorlessa · heitorlessa · commit db20c9b2f62b · 2021-07-23T20:39:39.000+02:00
diff --git a/aws_lambda_powertools/event_handler/api_gateway.py b/aws_lambda_powertools/event_handler/api_gateway.py
@@ -22,7 +22,7 @@
 
 _DYNAMIC_ROUTE_PATTERN = r"(<\w+>)"
 # Safe URI chars https://www.ietf.org/rfc/rfc3986.txt
-_NAMED_GROUP_BOUNDARY_PATTERN = r"(?P\1[-._~()'!*:@,;\\w]+)"
+_NAMED_GROUP_BOUNDARY_PATTERN = r"(?P\1[-._~()'!*:@%#,;\\w]+)"
 
 
 class ProxyEventType(Enum):
diff --git a/tests/functional/event_handler/test_api_gateway.py b/tests/functional/event_handler/test_api_gateway.py
@@ -703,7 +703,7 @@ def get_network_account(account_id: str, network_id: str):
     app.resolve(event, {})
 
 
-@pytest.mark.parametrize("req", [123456789, "user@example.com", "<foo>", "-._~'!*:@,;"])
+@pytest.mark.parametrize("req", [123456789, "user@example.com", "-._~'!*:@,;%#"])
 def test_non_word_chars_route(req):
     # GIVEN
     app = ApiGatewayResolver()

-Original file line number
+Diff line change
     app.resolve(event, {})
 -@pytest.mark.parametrize("req", [123456789, "[email protected]", "<foo>", "-._~'!*:@,;"])
 +@pytest.mark.parametrize("req", [123456789, "[email protected]", "-._~'!*:@,;%#"])
 def test_non_word_chars_route(req):
     # GIVEN
     app = ApiGatewayResolver()