mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-08 03:09:39 +03:00
[Bug Report]: OIDC Login #2474
Closed
opened 2026-02-05 04:16:00 +03:00 by OVERLORD
·
13 comments
No Branch/Tag Specified
development
further_theme_development
l10n_development
release
llm_only
vectors
v25-11
docker_env
drawio_rendering
user_permissions
ldap_host_failover
svg_image
prosemirror
captcha_example
fix/video-export
v25.12.3
v25.12.2
v25.12.1
v25.12
v25.11.6
v25.11.5
v25.11.4
v24.11.4
v25.11.3
v25.11.2
v25.11.1
v25.11
v25.07.3
v25.07.2
v25.07.1
v25.07
v25.05.2
v25.05.1
v25.05
v25.02.5
v25.02.4
v25.02.3
v25.02.2
v25.02.1
v25.02
v24.12.1
v24.12
v24.10.3
v24.10.2
v24.10.1
v24.10
v24.05.4
v24.05.3
v24.05.2
v24.05.1
v24.05
v24.02.3
v24.02.2
v24.02.1
v24.02
v23.12.3
v23.12.2
v23.12.1
v23.12
v23.10.4
v23.10.3
v23.10.2
v23.10.1
v23.10
v23.08.3
v23.08.2
v23.08.1
v23.08
v23.06.2
v23.06.1
v23.06
v23.05.2
v23.05.1
v23.05
v23.02.3
v23.02.2
v23.02.1
v23.02
v23.01.1
v23.01
v22.11.1
v22.11
v22.10.2
v22.10.1
v22.10
v22.09.1
v22.09
v22.07.3
v22.07.2
v22.07.1
v22.07
v22.06.2
v22.06.1
v22.06
v22.04.2
v22.04.1
v22.04
v22.03.1
v22.03
v22.02.3
v22.02.2
v22.02.1
v22.02
v21.12.5
v21.12.4
v21.12.3
v21.12.2
v21.12.1
v21.12
v21.11.3
v21.11.2
v21.11.1
v21.11
v21.10.3
v21.10.2
v21.10.1
v21.10
v21.08.6
v21.08.5
v21.08.4
v21.08.3
v21.08.2
v21.08.1
v21.08
v21.05.4
v21.05.3
v21.05.2
v21.05.1
v21.05
v21.04.6
v21.04.5
v21.04.4
v21.04.3
v21.04.2
v21.04.1
v21.04
v0.31.8
v0.31.7
v0.31.6
v0.31.5
v0.31.4
v0.31.3
v0.31.2
v0.31.1
v0.31.0
v0.30.7
v0.30.6
v0.30.5
v0.30.4
v0.30.3
v0.30.2
v0.30.1
v0.30.0
v0.29.3
v0.29.2
v0.29.1
v0.29.0
v0.28.3
v0.28.2
v0.28.1
v0.28.0
v0.27.5
v0.27.4
v0.27.3
v0.27.2
v0.27.1
v0.27
v0.26.4
v0.26.3
v0.26.2
v0.26.1
v0.26.0
v0.25.5
v0.25.4
v0.25.3
v0.25.2
v0.25.1
v0.25.0
v0.24.3
v0.24.2
v0.24.1
v0.24.0
v0.23.2
v0.23.1
v0.23.0
v0.22.0
v0.21.0
v0.20.3
v0.20.2
v0.20.1
v0.20.0
v0.19.0
v0.18.5
v0.18.4
v0.18.3
v0.18.2
v0.18.1
v0.18.0
v0.17.4
v0.17.3
v0.17.2
v0.17.1
v0.17.0
v0.16.3
v0.16.2
v0.16.1
v0.16.0
v0.15.3
v0.15.2
v0.15.1
v0.15.0
v0.14.3
v0.14.2
v0.14.1
v0.14.0
v0.13.1
v0.13.0
v0.12.2
v0.12.1
v0.12.0
v0.11.2
v0.11.1
v0.11.0
v0.10.0
v0.9.3
v0.9.2
v0.9.1
v0.9.0
v0.8.2
v0.8.1
v0.8.0
v0.7.6
v0.7.5
v0.7.4
v0.7.3
0.7.2
v.0.7.1
v0.7.0
v0.6.3
v0.6.2
v0.6.1
v0.6.0
v0.5.0
Labels
Clear labels
🎨 Design
📖 Docs Update
🐛 Bug
🐛 Bug
:cat2:🐈 Possible duplicate
💿 Database
☕ Open to discussion
💻 Front-End
🐕 Support
🚪 Authentication
🌍 Translations
🔌 API Task
🏭 Back-End
⛲ Upstream
🔨 Feature Request
🛠️ Enhancement
🛠️ Enhancement
🛠️ Enhancement
❤️ Happy feedback
🔒 Security
🔍 Pending Validation
💆 UX
📝 WYSIWYG Editor
🌔 Out of scope
🔩 API Request
:octocat: Admin/Meta
🖌️ View Customization
❓ Question
🚀 Priority
🛡️ Blocked
🚚 Export System
♿ A11y
🔧 Maintenance
> Markdown Editor
pull-request
Mirrored from GitHub Pull Request
No Label
🐛 Bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/BookStack#2474
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
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?
Originally created by @git-noise on GitHub (Nov 1, 2021).
Describe the Bug
OIDC login leads to a blank page and a HTTP 500 query answer
Steps to Reproduce
Expected Behaviour
OIDC Login should redirect to the OIDC provider login page or log the user into Bookstack if already logged-in
Screenshots or Additional Context
Laravel logs:
Exact BookStack Version
v21.10.3 - same on v21.10.2
PHP Version
7.2
Hosting Environment
Ubuntu 18.04 LTS, Apachee2 reverse-proxy, manual installation.
@ssddanbrown commented on GitHub (Nov 1, 2021):
Hi @git-noise,
Could you open the file
/opt/bookstack/BookStack/storage/framework/views/1a15a2c52c34a48c83eaeee648b7bab225c2ed2d.phpon your instance, scroll to the bottom and find a line that contains "PATH" and "ENDPATH" then provide the show path? I just need to understand where this may be originating from.@git-noise commented on GitHub (Nov 1, 2021):
Hello and thanks for your answer.
Sure thing it is:
/opt/bookstack/BookStack/resources/views/layouts/base.blade.phpThanks,
@ssddanbrown commented on GitHub (Nov 1, 2021):
Thanks @git-noise, Not really sure why you'd be getting that error.
Could you:
git statuswithin the BookStack install folder.@git-noise commented on GitHub (Nov 2, 2021):
Here it is:
This change comes from mysql TLS config
URL path:
/oidc/loginStack trace:
Many thanks,
@ssddanbrown commented on GitHub (Nov 2, 2021):
@git-noise Thanks for providing the information but I really don't understand this one.
The
/oidc/loginendpoint should not even utilise a view which is what the system is attempting use here and erroring out upon. Additionally, the cspNonce value should be available to all web endpoint views.Does everything in a non-oidc context work without issue? So with standard authentication can you access the system and perform all common functions without issue?
@git-noise commented on GitHub (Nov 2, 2021):
@ssddanbrown
Indeed I can, either via local database or gitlab and everything works well.
Regarding OIDC case, nothing in the web console either, except for that 500 internal error to https://mydomain/oidc/login.
That being said, I am getting a 404 on that /oidc/login when I am doing a get on it. Am I redirected to the right page?
@ssddanbrown commented on GitHub (Nov 2, 2021):
404 on
GETis expected, since it's a post endpoint.Can you list any
.phpfiles that are in thebootstrap/cachedirectory within your instance?@git-noise commented on GitHub (Nov 2, 2021):
I figured, makes sense, but you never know
Yes so I have:
@git-noise commented on GitHub (Nov 3, 2021):
@ssddanbrown
Hello,
If it makes sense I'll try to redeploy things in a containerized environment and report back. If I am the only one with this error it may be linked to some legacy things from previous installation that are not playing too well. It won't be immediate, but I'll get to it when I can.
Best,
@santiago-mooser commented on GitHub (Nov 8, 2021):
I'm having the same issue, but I think it may have to do with a broken foreign key constraint in my database. I upgraded from MariaDB 10.4.x to 10.5.12 and bookstack 21.04.2 to 21.08.5 and I'm getting a 500 error when I try to sign in.
The setup I'm currently running is using docker-compose:
And I made the mistake of upgrading both containers without checking whether I needed to go through an upgrade procedure (my mistake...), so I'm in the process of figuring out what the problem is.
This is the nginx error log:
And this is the php error log (there doesn't seem to be anything unusual here):
However, I am getting some weird errors with MariaDB when I try to export it with mysqldump:
Considering that someone got a similar error when there was a piece of data that broke a foreign key contraint, I think something happened in the update from 21.04.2 to 21.08.6 that fucked a database row up and is now preventing the database from properly working.
I'll look into it a bit more and update as soon as I figure out the problem.
@santiago-mooser commented on GitHub (Nov 8, 2021):
So, it seems that there is some type of corruption or attribute break in the
activitiestable. I was able to figure out by manually dumping all the different tables.I got the full table list:
Then wrote up a really dirty mysqldump script:
This would crash my database(?) which would then refuse to do anything:
From here, I simply removed the
activitiestable from the dump script:And was able to successfully extract all other tables:
Conclusion
Something in the upgrade process of either the database or bookstack corrupted or changed the values in the
activitiestable to the point where not even the DBMS is able to open the table correctly. I'll have to see whether I'm able to re-import the information without theactivitiestable into a new bookstack instance.I have no idea what happened but I'll update you guys as I figure it out 🤷🏼
@git-noise commented on GitHub (Nov 8, 2021):
Interesting @santiago-mooser. I am not using mariadb and do not have any DB issues afaik, but that maybe points to some upgrade silently gone wrong then.
@ssddanbrown commented on GitHub (Apr 25, 2022):
Since there's been no recent activity here I'm going to close this. If the issue remains and is something you still require to be fixed respond to my comment with the current status and this can then be re-opened.