SAML existing user different credentials error may not be notifying #1857

New Issue

OVERLORD · 2026-02-05T02:03:46+03:00

OVERLORD commented

2026-02-05 02:03:46 +03:00

Originally created by @ssddanbrown on GitHub (Sep 22, 2020).

As discovered providing support in discord, The getOrRegister action here may lead to this exception being thrown which may not be notifying. Need to check and handle that case if so.

https://discordapp.com/channels/578552496637739008/585101285112676392/757741989767676044

Originally created by @ssddanbrown on GitHub (Sep 22, 2020). As discovered providing support in discord, The getOrRegister [action here](https://github.com/BookStackApp/BookStack/blob/87a5340a0526d4f46d4da676ce44f4e26039d281/app/Auth/Access/Saml2Service.php#L364) may lead to [this exception](https://github.com/BookStackApp/BookStack/blob/87a5340a0526d4f46d4da676ce44f4e26039d281/app/Auth/Access/RegistrationService.php#L60) being thrown which may not be notifying. Need to check and handle that case if so. https://discordapp.com/channels/578552496637739008/585101285112676392/757741989767676044

OVERLORD added the 🐛 Bug 🔍 Pending Validation labels 2026-02-05 02:03:46 +03:00

OVERLORD closed this issue

2026-02-05 02:03:51 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#1857