Skip to content

clear LDAP cache after user deletion - #21318

Merged
rullzer merged 1 commit into
masterfrom
fix/noid/user-delete-clear-ldap-cache
Jun 9, 2020
Merged

clear LDAP cache after user deletion#21318
rullzer merged 1 commit into
masterfrom
fix/noid/user-delete-clear-ldap-cache

Conversation

@blizzz

Copy link
Copy Markdown
Member

Before:

$ php occ user:delete person_133176
The specified user was deleted
$ php occ user:info 'person_133176' - user_id: person_133176
- display_name: Iris, Burns (person_133176)
- email: - cloud_id: person_133176@nc.zara/master
- enabled: true
- groups:
- quota: none
- last_seen: 1970-01-01T00:00:00+00:00
- user_directory: /path/to/data/person_133176
- backend: LDAP

(^ the data is even incomplete, e.g. no email)

Now:

$ php occ user:delete person_133176
The specified user was deleted
$ php occ user:info 'person_133176'
user not found

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzzblizzz added this to the Nextcloud 20 milestone Jun 8, 2020
@blizzz

Copy link
Copy Markdown
MemberAuthor

/backport to stable19

@blizzz

Copy link
Copy Markdown
MemberAuthor

/backport to stable18

@blizzz

Copy link
Copy Markdown
MemberAuthor

/backport to stable17

@ChristophWurstChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@blizzz

Copy link
Copy Markdown
MemberAuthor

https://www.martinfowler.com/bliki/TwoHardThings.html wink

This is incomplete. He forgot to mention off-by-one errors. At least the title is correct.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: ldap

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@blizzz@rullzer@ChristophWurst