starred/pocket-id-pocket-id-1
1
0
Fork 0
mirror of https://github.com/pocket-id/pocket-id.git synced 2025-12-11 07:42:58 +03:00
Code Issues 122 Packages Projects Releases 10 Wiki Activity

🐛 Bug Report: removed ldap users aren't deleted in pocket id #273

New Issue
Closed
opened 2025-10-08 00:00:38 +03:00 by OVERLORD · 4 comments
OVERLORD commented 2025-10-08 00:00:38 +03:00
Owner
Copy Link

Originally created by @tupcakes on GitHub.

Originally assigned to: @kmendell on GitHub.

Reproduction steps

I noticed when deleting a user in ldap (using lldap) that the pocket id sync will not delete the user.

Expected behavior

I wasn't sure if this was intended, but it seems like deleting a user in ldap should remove from pocket id, or at least have a way in the pocket id gui to manually delete the user.

Actual Behavior

User was not deleted.

Version and Environment

0.44.0
Environment: k3s
Proxy: ingress-nginx

Log Output

This is the only event in the log related to the operation:
2025/03/27 19:43:43 Failed to delete user svcauthentik-sync with: LDAP users can't be updated

Originally created by @tupcakes on GitHub. Originally assigned to: @kmendell on GitHub. ### Reproduction steps I noticed when deleting a user in ldap (using lldap) that the pocket id sync will not delete the user. ### Expected behavior I wasn't sure if this was intended, but it seems like deleting a user in ldap should remove from pocket id, or at least have a way in the pocket id gui to manually delete the user. ### Actual Behavior User was not deleted. ### Version and Environment 0.44.0 Environment: k3s Proxy: ingress-nginx ### Log Output This is the only event in the log related to the operation: `2025/03/27 19:43:43 Failed to delete user svcauthentik-sync with: LDAP users can't be updated`
OVERLORD added the bug label 2025-10-08 00:00:38 +03:00
OVERLORD commented 2025-10-08 00:00:43 +03:00
Author
Owner
Copy Link

@kmendell commented on GitHub:

@tupcakes Can you try this image ghcr.io/kmendell/pocket-id:delete-ldap-users and see if the issue is fixed?

@kmendell commented on GitHub: @tupcakes Can you try this image `ghcr.io/kmendell/pocket-id:delete-ldap-users` and see if the issue is fixed?
OVERLORD commented 2025-10-08 00:00:44 +03:00
Author
Owner
Copy Link

@tupcakes commented on GitHub:

Seems to work. Nice! Thank you.

@tupcakes commented on GitHub: Seems to work. Nice! Thank you.
OVERLORD commented 2025-10-08 00:00:46 +03:00
Author
Owner
Copy Link

@kmendell commented on GitHub:

Fixed in: 7e658276f0

@kmendell commented on GitHub: Fixed in: https://github.com/pocket-id/pocket-id/commit/7e658276f04d08a1f5117796e55d45e310204dab
OVERLORD commented 2025-10-08 00:00:47 +03:00
Author
Owner
Copy Link

@kmendell commented on GitHub:

I'll take a look at this. Thank you for reporting!

@kmendell commented on GitHub: I'll take a look at this. Thank you for reporting!
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
breaking
bug
documentation
feature
needs more upvotes
open to pull requests
pull-request
Mirrored from GitHub Pull Request
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-1#273
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?