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

Microsoft Graph Attribute raw_info['mail'] causes error #6

Open
iiiich opened this issue Apr 7, 2020 · 0 comments
Open

Microsoft Graph Attribute raw_info['mail'] causes error #6

iiiich opened this issue Apr 7, 2020 · 0 comments

Comments

@iiiich
Copy link

iiiich commented Apr 7, 2020

Hi Blinsidenetworks-Team,

we got an on-prem Active Directory which is synced to our Azure-AD and try to connect our Azure-AD to bigbluebutton. It seems to be a problem when the https://graph.microsoft.com/v1.0/me/ endpoint delivers a mail-attribute.
If we use a normal Azure-AD-User the authentication works well, but the synced users from our Active Directory with the mail-attribute can't login and get the error "Invalid_credentials".

I looked into the code and maybe the following lines in
https://github.com/blindsidenetworks/omniauth-bn-office365/blob/master/lib/omniauth/strategies/office365.rb causes the error:

…
36:    email: raw_info['mail'] || raw_info['userPrincipalName']
…
91:     email = token["mail"] || token["userPrincipalName"]
…

Edit: After some changes we could deliver the on-prem users without the email-field, but they still can't login.

Thank you very much,
robert

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