-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat(apigateway): lambda request authorizer #5642
Conversation
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix the build and branch conflicts. I'd like to check the PR out and give it a whirl.
packages/@aws-cdk/aws-apigateway/test/authorizers/test.lambda.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, update the README.md with information about request authorizer.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
@nija-at Not really sure what to do about the build failures, it seems like a JSII issue, not a TS one (for instance, my IDE has no issue with the code as is). I think I've pared down and simplified things as much as possible otherwise. |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
9616f63
to
d1838be
Compare
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
0032340
to
8c6c839
Compare
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
@nija-at should be ready for final review now 👍 |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry about the multiple iterations. This is very close...
packages/@aws-cdk/aws-apigateway/lib/authorizers/identity-source.ts
Outdated
Show resolved
Hide resolved
packages/@aws-cdk/aws-apigateway/lib/authorizers/identity-source.ts
Outdated
Show resolved
Hide resolved
packages/@aws-cdk/aws-apigateway/lib/authorizers/identity-source.ts
Outdated
Show resolved
Hide resolved
This creates a common LambdaAuthorizer base class so that the token and request authorizers can share common functionality.
Pull request has been modified.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request is now being automatically merged. |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request is now being automatically merged. |
First of all, thank you @CaerusKaru for the contribution! My team had just realized last week that request authorizers weren't supported yet and were considering contributing ourselves. Needless to say, we were pleasantly surprised seeing you were already working on this. Secondly, what is the publish cycle for the aws-cdk package? Really looking forward to this addition. |
Our publish cycle is weekly, so you should likely expect this to be available later this week. If, for some reason, we can't get it out this week, it will be available next week. Thanks! |
Many thanks to @nija-at for sticking with me on this PR. If only things could be as simple as you think they are when you start 😉 |
Commit Message
feat(apigateway): lambda request authorizer (#5642)
This creates a common LambdaAuthorizer base class so that the
token and request authorizers can share common functionality.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license