-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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(cognito): add PreTokenGeneration lambda trigger support #3910
Conversation
Pull Request Checklist
|
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
Codebuild (Continuous Integration) build failed for current commits. Please check log and resolve before PR is merged. |
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.
Thanks for the pull request. Apologies for the delay in getting to this.
Could you also similarly include the UserMigration trigger?
Pull request has been modified.
…r pool test(if api correctly appends triggers)
Pull request has been modified.
@@ -53,34 +53,93 @@ export = { | |||
'on* API correctly appends triggers'(test: Test) { | |||
// GIVEN | |||
const stack = new cdk.Stack(); | |||
const fn = new lambda.Function(stack, 'MyLambda', { | |||
|
|||
const createAuthChallengeLambdaFn = new lambda.Function(stack, 'createAuthChallengeLambda', { |
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.
Would've been cleaner to refactor the new lambda.Function(...)
into a separate function to reduce code repeat.
Approving nevertheless.
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.
You can open an issue regarding cognition tests and assign it to me. :) We can have a discussion there of what is missing and how we can improve those tests.
Thank you for contributing! Your pull request is now being automatically merged. |
1 similar comment
Thank you for contributing! Your pull request is now being automatically merged. |
fixes #2497
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license