Bug Fix: Update Cognito Authentication Flow #672
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix: Update Cognito Authentication Flow for Federated Users
Summary
This PR updates the Cognito authentication flow for federated users by replacing the POST_AUTHENTICATION trigger with PRE_SIGN_UP and POST_CONFIRMATION triggers. This change resolves the issue where users were experiencing 404 errors after logout when using Cognito federation.
Problem
When using Cognito federation, the previous implementation relied on the POST_AUTHENTICATION trigger to assign users to groups. However, this approach caused issues where users would get 404 errors after logout, and the Lambda function would only trigger after the user had already logged into the console.
Solution
The solution implements a more robust authentication flow by:
This approach ensures that users are properly assigned to groups earlier in the authentication flow, preventing the 404 errors that occurred with the previous implementation.
Changes
Modified
lib/authentication/index.ts
to:Enhanced
lib/authentication/lambda/addFederatedUserToUserGroup/index.py
to:Testing
The changes have been tested with Cognito federation enabled, confirming that: