-
Notifications
You must be signed in to change notification settings - Fork 367
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
add configurable username claim #280
Conversation
Thanks for the quick turn around! This PR looks ready to go after my double check comment. I'd merge this as is and then we can have a second PR to switch over |
Looks like the fallback is to |
Sorry, should have read through better. Thanks! |
No worries, and thanks for merging! |
This is a suggestion to fix #213 that is similar to #224, but provides a configurable
username_claim
attribute.Logic:
c.AzureAdOAuthenticator.username_claim
name if config isn't setoid
closes #224