Issue #3 edit apigw auth identity_source comment specify header requi…

…rement Former-commit-id: 2e15f5b
VerticalRelevance · Apr 28, 2022 · 23a4dc3 · 23a4dc3
1 parent 33ed1cc
commit 23a4dc3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/stacks/client_stack.py b/stacks/client_stack.py
@@ -78,9 +78,7 @@ def apigw(self):
             response_types=[aws_apigatewayv2_authorizers_alpha.HttpLambdaResponseType.SIMPLE],
             results_cache_ttl = Duration.seconds(0),
             identity_source = [
-                "$request.header.Authorization", # request must match or 401: requests.get(invoke_url,headers={'Authorization':'foo'})
-                # "$context.identity.principalOrgId",
-            ]
+                "$request.header.Authorization", # Authorization my be present in headers or 401, e.g. r = requests.post(url,auth = auth, ...)
         )
 
         # auth - iam
@@ -141,7 +139,7 @@ def apigw(self):
             # authorizer=authorizer_iam
         )
 
-        self.apigw_full_invoke_url = f'{self.http_api.url}{self.path}'
+        self.apigw_full_invoke_url = f'{self.http_api.url[:-1]}{self.path}' # remove duplicate slash
 
         CfnOutput(self, "ApigwInvokeUrl", value=self.apigw_full_invoke_url)
 

diff --git a/supplementary_files/lambdas/sign_apigw_request/lambda_function.py b/supplementary_files/lambdas/sign_apigw_request/lambda_function.py
@@ -32,8 +32,6 @@ def lambda_handler(event,context):
 
     print(f'BotoAWSRequestsAuth:\n{auth}')
 
-    # headers = {'Authorization':'foo'}
-
     control_broker_consumer_input = event
 
     r = requests.post(