Skip to content

feat: #348 unify auth identifiers#371

Open
goralczyks wants to merge 14 commits intodevelopfrom
feature/#348-unify-auth-identifiers
Open

feat: #348 unify auth identifiers#371
goralczyks wants to merge 14 commits intodevelopfrom
feature/#348-unify-auth-identifiers

Conversation

@goralczyks
Copy link
Collaborator

@goralczyks goralczyks commented Jan 22, 2026

📃 Ticket

#348

✍ Description

  1. I've added user_identifier property for User, changed username/name to display_name for User and Whitelist User. OAS and DB have been updated accordingly
  2. New user uses email for singn-up, email is stored also as user_identifier propery. lambda_auth, middlewares and routers refer to user_identifier for user auth and identification whenever applicable.

@goralczyks goralczyks self-assigned this Jan 22, 2026
@goralczyks goralczyks linked an issue Jan 22, 2026 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Jan 22, 2026

@github-actions github-actions bot added the feature Adding a new feature label Jan 22, 2026
@github-actions
Copy link
Contributor

🚨 Potential Breaking Changes Detected: admin API 🚨

7 changes: 0 error, 7 warning, 0 info

🔍 View Detailed GitHub Action Log

Last updated: Fri, 23 Jan 2026 10:43:12 GMT

@goralczyks goralczyks marked this pull request as ready for review January 23, 2026 11:00
Copy link
Contributor

@shgokita shgokita left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Adding a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task]: Mixed Authentication Identifiers: Email vs Username

3 participants