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

SAML new username issue #14895

Closed
makhoff opened this issue Jun 28, 2019 · 2 comments · Fixed by #18213
Closed

SAML new username issue #14895

makhoff opened this issue Jun 28, 2019 · 2 comments · Fixed by #18213

Comments

@makhoff
Copy link

makhoff commented Jun 28, 2019

Description:

Hello.
When I configured SAML with settings from this issue, new user created in format "Display Name, login" and I can't manage him from RocketChat console(only via REST API) .

In logs I see that parameter username received as "username":["XXX, YYY","TSTDMN\xxx.yyy"]" instead of ""username":["XXX, YYY"]"

Steps to reproduce:

  1. Configure SAML with ADFS
  2. Set "Generate Username" = false
  3. Sign in with new SSO user

Expected behavior:

Got username in format ""username":["XXX, YYY"]"

Actual behavior:

rch1
rch2

Server Setup Information:

  • Version of Rocket.Chat Server: 1.2.0
  • Operating System: RHEL 7
  • Deployment Method: on premise
  • Number of Running Instances: 1
  • DB Replicaset Oplog: Enabled
  • NodeJS Version: 8.11.4
  • MongoDB Version: 4.0.10

Additional context

Relevant logs:

RESULT :{"profile":{"inResponseToId":"id-JPKNNNsuCovo5t8CD","issuer":"http://MYDOMAIN.com/adfs/services/trust","nameID":"[email protected]","nameIDFormat":"urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress","sessionIndex":"_921934bf-672a-4364-b281-432732f36c6e",**"username":["DISPLAY NAME","DOMAIN\sAMAccountaname"],"**email":"MY_EMAIL","http://schemas-xmlsoap-org/ws/2005/05/identity/claims/name":"DISPLAY NAME",

@routerino
Copy link

Any progress on this? Encountering this identical issue and it's making me tear my hair out.

@1rV1N-git
Copy link
Contributor

@makhoff Change in this place to "cn"
image

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

Successfully merging a pull request may close this issue.

4 participants