400 interaction_required #1540

Closed
opened 2026-02-05 01:11:33 +03:00 by OVERLORD · 3 comments
Owner

Originally created by @ch0wm3in on GitHub (Feb 16, 2020).

Describe the bug
When using socialite provider for azure + Conditonal access with MFA, it will not allow MFA through bookstack.

Steps To Reproduce
Steps to reproduce the behavior:

  1. Click Login With Microsoft
  2. Have MFA enabled for organisation in Azure
  3. See error - Which is unknown error in bookstack and 400 interaction_required in laravel log.

Expected behavior
To be logged into the application

Screenshots
If applicable, add screenshots to help explain your problem.

Your Configuration (please complete the following information):

  • 28.2
  • php7-7.3.14-r0
  • Docker

Additional context
Please see my pull request which fixes above, it is a fix recommended by socialite provider.
https://github.com/BookStackApp/BookStack/pull/1889#issue-374326205

Originally created by @ch0wm3in on GitHub (Feb 16, 2020). **Describe the bug** When using socialite provider for azure + Conditonal access with MFA, it will not allow MFA through bookstack. **Steps To Reproduce** Steps to reproduce the behavior: 1. Click Login With Microsoft 2. Have MFA enabled for organisation in Azure 3. See error - Which is unknown error in bookstack and 400 interaction_required in laravel log. **Expected behavior** To be logged into the application **Screenshots** If applicable, add screenshots to help explain your problem. **Your Configuration (please complete the following information):** - 28.2 - php7-7.3.14-r0 - Docker **Additional context** Please see my pull request which fixes above, it is a fix recommended by socialite provider. https://github.com/BookStackApp/BookStack/pull/1889#issue-374326205
OVERLORD added the 🚪 Authentication label 2026-02-05 01:11:33 +03:00
Author
Owner

@ch0wm3in commented on GitHub (Mar 3, 2020):

@ssddanbrown Any plans for merging?

@ch0wm3in commented on GitHub (Mar 3, 2020): @ssddanbrown Any plans for merging?
Author
Owner

@ssddanbrown commented on GitHub (Mar 3, 2020):

@ch0wm3in Yes, I do plan to. Just haven't got round to it yet as I'd need to spend some time understanding how it fixes the issue and if it will possibly cause issues for existing setups.

@ssddanbrown commented on GitHub (Mar 3, 2020): @ch0wm3in Yes, I do plan to. Just haven't got round to it yet as I'd need to spend some time understanding how it fixes the issue and if it will possibly cause issues for existing setups.
Author
Owner

@ssddanbrown commented on GitHub (Mar 6, 2020):

As mentioned on #1889, Tested on my dev instance, which did not have this issue, and did not cause any side-effects there (Thinking of existing users), so now merged to be part of the next patch release.

Thanks for reporting and fixing @ch0wm3in.

@ssddanbrown commented on GitHub (Mar 6, 2020): As mentioned on #1889, Tested on my dev instance, which did not have this issue, and did not cause any side-effects there (Thinking of existing users), so now merged to be part of the next patch release. Thanks for reporting and fixing @ch0wm3in.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#1540