Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update apigatewayv2 request context #328

Merged
merged 2 commits into from Oct 5, 2020
Merged

Conversation

Tolledo
Copy link
Contributor

@Tolledo Tolledo commented Oct 1, 2020

Description of changes:
On Sept 9th, API Gateway launched support for customers to secure Amazon API Gateway HTTP APIs using two new authorization options: Lambda authorizers and IAM authorizers. These new options enable customers to make flexible authorization decisions by providing an AWS Lambda function, or leveraging AWS IAM policies to control access to their APIs without writing any code.

Updating API Gateway v2 event to support new context variables.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov-commenter
Copy link

codecov-commenter commented Oct 1, 2020

Codecov Report

Merging #328 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #328   +/-   ##
=======================================
  Coverage   72.46%   72.46%           
=======================================
  Files          18       18           
  Lines         730      730           
=======================================
  Hits          529      529           
  Misses        136      136           
  Partials       65       65           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b93d48...855955a. Read the comment docs.

Copy link
Collaborator

@bmoffatt bmoffatt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few naming conventions need to be fixed. One open question about the Lambda authorizer

events/apigw.go Outdated Show resolved Hide resolved
events/apigw.go Outdated Show resolved Hide resolved
events/apigw.go Outdated Show resolved Hide resolved
events/apigw.go Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants