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

feat(data_classes): Cognito custom auth triggers #178

Merged
merged 8 commits into from
Oct 2, 2020

Conversation

michaelbrewer
Copy link
Contributor

@michaelbrewer michaelbrewer commented Sep 24, 2020

Issue #, if available:

Description of changes:

Add support for "Custom Authentication Challenge Lambda Triggers":

  • Add response helpers
  • Implement "Define auth challenge", "Create auth challenge" and "Verify
    auth challenge response"

Checklist

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

Michael Brewer added 3 commits September 24, 2020 13:30
Add support for "Custom Authentication Challenge Lambda Triggers":
* Add response helpers
* Implement "Define auth challenge", "Create auth challenge" and "Verify
  auth challenge response"
@codecov-commenter
Copy link

codecov-commenter commented Sep 24, 2020

Codecov Report

Merging #178 into develop will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##           develop     #178    +/-   ##
=========================================
  Coverage    99.86%   99.86%            
=========================================
  Files           52       52            
  Lines         2154     2275   +121     
  Branches        97       99     +2     
=========================================
+ Hits          2151     2272   +121     
  Misses           3        3            
Impacted Files Coverage Δ
...bda_powertools/utilities/data_classes/ses_event.py 100.00% <ø> (ø)
.../utilities/data_classes/cognito_user_pool_event.py 100.00% <100.00%> (ø)

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 d08de0b...802324f. Read the comment docs.

Copy link

@Nr18 Nr18 left a comment

Choose a reason for hiding this comment

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

Spotted a typo

@heitorlessa
Copy link
Contributor

dev builds fixed - please rebase and apologies for the inconvenience.

https://github.com/awslabs/aws-lambda-powertools-python/runs/1165140290?check_suite_focus=true

Copy link
Contributor

@heitorlessa heitorlessa left a comment

Choose a reason for hiding this comment

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

LGTM - Thanks a lot for those changes, and other minor fixes in docstrings I noticed @michaelbrewer

Merging it..

@heitorlessa heitorlessa merged commit 9661f13 into aws-powertools:develop Oct 2, 2020
@michaelbrewer michaelbrewer deleted the cognito-custom-auth branch February 22, 2021 13:18
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.

4 participants