fix: prevent deadlock when trying to delete LDAP users #227

New Issue

AtHeartEngineer · 2025-07-08T08:42:57-04:00

AtHeartEngineer commented

2025-07-08 08:42:57 -04:00

Originally created by @ItalyPaleAle on 4/22/2025

Fixup from #437
Reverted LDAP to use deleteUserInternal which accepts an existing transaction. Without that, it would cause a deadlock in SQLite, as we can't start a transaction if there's another one active already

Also renamed DisableUser to disableUserInternal since it's another "internal" method

*Originally created by @ItalyPaleAle on 4/22/2025* Fixup from #437 Reverted LDAP to use `deleteUserInternal` which accepts an existing transaction. Without that, it would cause a deadlock in SQLite, as we can't start a transaction if there's another one active already Also renamed `DisableUser` to `disableUserInternal` since it's another "internal" method

AtHeartEngineer closed this issue

2025-07-08 08:42:57 -04:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/pocket-id#227