Skip to content

Comments

feature/(auth): Handle users with invalid email addresses in OAuth ca…#13

Open
hongwei1 wants to merge 1 commit intoOpenBankProject:mainfrom
hongwei1:main
Open

feature/(auth): Handle users with invalid email addresses in OAuth ca…#13
hongwei1 wants to merge 1 commit intoOpenBankProject:mainfrom
hongwei1:main

Conversation

@hongwei1
Copy link
Collaborator

…llback

  • Remove email requirement from user validation check, only require user_id
  • Add validation logic to detect placeholder email addresses (@noemail.local)
  • Log warning when users login without valid email addresses
  • Include commented code for optional redirect to profile page for email updates
  • Update error message to reflect that only user_id is required for validation
  • Allows users with placeholder emails to proceed with session creation while flagging for potential email update

…llback

- Remove email requirement from user validation check, only require user_id
- Add validation logic to detect placeholder email addresses (@noemail.local)
- Log warning when users login without valid email addresses
- Include commented code for optional redirect to profile page for email updates
- Update error message to reflect that only user_id is required for validation
- Allows users with placeholder emails to proceed with session creation while flagging for potential email update
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.

1 participant