-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
LDAP: announce display name changes so that addressbook picks it up #12054
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code makes sense 👍
Causes UPDATE: I have a plausible theory now. This is corndered by some setup bits: 5 tables should not contain data about the certain user, cron mode must be ajax and no memcache must be set up. could reproduce. |
Should do now. Happy about the integration tests catching this beast :) |
CI is not happy |
… when an unmapped user logs in for the first time when background job mode is ajax and no memcache was configured. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
9adf542
to
49456e4
Compare
Indeed, I missed to adjust the tests with the last commit. |
only write when the displayname differs, but then announce it refs #5212 and fixes #9112 Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de> do not run into UniqueConstraintViolationException … when an unmapped user logs in for the first time when background job mode is ajax and no memcache was configured. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
only write when the displayname differs, but then announce it refs #5212 and fixes #9112 Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de> do not run into UniqueConstraintViolationException … when an unmapped user logs in for the first time when background job mode is ajax and no memcache was configured. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
fixes #9112 (and refs #5212)
Test is:
Now the new displayname should be presented, previously it did not.
@nextcloud/ldap