Skip to content

Conversation

dalumiller
Copy link
Contributor

Your SAM Description says this is the "API Gateway with Lambda Token Authorizer" when, I believe, this is a Request Authorizer. Your API resource FunctionPayloadType says "REQUEST" and the link below seems to be the Token Authorizer example.

https://github.com/awslabs/serverless-application-model/blob/master/examples/2016-10-31/api_lambda_token_auth/template.yaml

Issue #, if available:

Description of changes:

Description of how you validated changes:

Checklist:

  • Write/update tests
  • make pr passes
  • Update documentation
  • Verify transformed template deploys and application functions as expected
  • Add/update example to examples/2016-10-31

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Your SAM Description says this is the "API Gateway with Lambda Token Authorizer" when, I believe, this is a Request Authorizer. Your API resource FunctionPayloadType says "REQUEST" and the link below seems to be the Token Authorizer example.

https://github.com/awslabs/serverless-application-model/blob/master/examples/2016-10-31/api_lambda_token_auth/template.yaml
@jlhood jlhood changed the base branch from release/v1.8.0 to develop August 13, 2019 17:20
@jlhood jlhood closed this Aug 13, 2019
@jlhood jlhood reopened this Aug 13, 2019
@jlhood
Copy link
Contributor

jlhood commented Aug 13, 2019

Change was made against the wrong base branch. Changed to develop and closed/reopened to kick off another Travis build.

@jlhood jlhood requested a review from praneetap August 13, 2019 17:21
@codecov-io
Copy link

codecov-io commented Aug 13, 2019

Codecov Report

Merging #1068 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1068   +/-   ##
========================================
  Coverage    94.81%   94.81%           
========================================
  Files           69       69           
  Lines         3276     3276           
  Branches       639      639           
========================================
  Hits          3106     3106           
  Misses          89       89           
  Partials        81       81

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 f61e707...18debf5. Read the comment docs.

@jlhood jlhood requested review from keetonian and removed request for praneetap August 13, 2019 18:22
Copy link
Contributor

@praneetap praneetap left a comment

Choose a reason for hiding this comment

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

Cool, thanks for catching this! And thanks for your contribution :)

@praneetap praneetap merged commit 2aff2db into aws:develop Aug 14, 2019
@praneetap praneetap mentioned this pull request Sep 19, 2019
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.

5 participants