RocketChat / RocketChat/Rocket.Chat

LDAP doesn't update users created by Oauth

Open
#14,409 2 comments 0 reactions 1 assignee View on GitHub

Nobody has claimed this yet.

area: authentication
Dominant language
TypeScript
Stars
46.1k
Forks
13.9k
Avg merge
3d 3h
Merged PRs (30d)
130

Description

When I do a manual background sync of LDAP, I get this error with users created by manual oauth.
It should update the users with the LDAP password so I can login using LDAP as an alternative, because Oauth is not working in any of the Android clients, only iOS.

34mI20190507-09:18:36.291(0) server.js:199 LDAPSync ➔ error Error creating user { Error: Email already exists. [403] at handleError (packages/accounts-password/password_server.js:104:17) at checkForCaseInsensitiveDuplicates (packages/accounts-password/password_server.js:245:7) at createUser (packages/accounts-password/password_server.js:1076:3) at AccountsServer.Accounts.createUser (packages/accounts-password/password_server.js:1147:10) at addLdapUser (app/ldap/server/sync.js:260:29) at ldapUsers.forEach.ldapUser (app/ldap/server/sync.js:323:5) at Array.forEach () at ldap.searchUsersSync.Meteor.bindEnvironment (app/ldap/server/sync.js:290:13) at runWithEnvironment (packages/meteor.js:1356:24) isClientSafe: true, error: 403, reason: 'Email already exists.', details: undefined, message: 'Email already exists. [403]', errorType: 'Meteor.Error' }

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.