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

Store+return email field in addition to user_name #29

Open
theferrit32 opened this issue Jan 10, 2019 · 4 comments
Open

Store+return email field in addition to user_name #29

theferrit32 opened this issue Jan 10, 2019 · 4 comments
Assignees

Comments

@theferrit32
Copy link
Member

Some use cases rely on there being an associated email address with a given identity, and the user_name is not sufficient. Since most identity providers include an associated email address already, look for this in the token response, and if present, store it and return it to clients

@theferrit32
Copy link
Member Author

This requires changes because some services don't use email as the identity. Some use a [email protected] but have it be something that is not valid email address (example: Harvard, UNC-CH).

@theferrit32 theferrit32 self-assigned this Jan 10, 2019
@theferrit32
Copy link
Member Author

User model will be updated to include an email field, which requires django database migrations.

Value of email field will default to null/None

@theferrit32
Copy link
Member Author

Value of email field will default to empty string

@theferrit32
Copy link
Member Author

Value should be added to /token response

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

No branches or pull requests

1 participant