mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-08 03:09:39 +03:00
Enabling LDAP lost admin access #1649
Closed
opened 2026-02-05 01:30:28 +03:00 by OVERLORD
·
7 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
Milestone
No items
No Milestone
Projects
Clear projects
No project
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/BookStack#1649
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 @kabaga on GitHub (Apr 14, 2020).
Describe the bug
According the Bookstackapp instruction, to enable LDAP, the LDAP parameters need to be added to the .env file. The issue that I am having is when I login to the web UI using the local Bookstack admin account, then change the AUTH_METHOD from standard to ldap. Bookstack logs me out. When I try to login again, Bookstack gives me an error stating "These credentials do not match our records."
LDAP works because I can login with my ldap user account, but no admin privilege.
I couldn't get the LDAPS to work at all.
Steps To Reproduce
Steps to reproduce the behavior:
Expected behavior
Step 6 should not kick me out and should allow me to continue working on ‘External Authentication ID’
Screenshots
n/a
Your Configuration (please complete the following information):
Additional context
I am using Univention as my LDAP server. Here is the .env LDAP parameters.
AUTH_METHOD=ldap
LDAP_SERVER=10.0.7.40:7389
#LDAP_SERVER=ldaps://10.0.7.40:7636
LDAP_BASE_DN=dc=example,dc=com
LDAP_DN=uid=bookstack,cn=users,dc=example,dc=com
LDAP_PASS=password
LDAP_USER_FILTER=(&(uid=${user}))
LDAP_VERSION=3
LDAP_ID_ATTRIBUTE=uid
LDAP_EMAIL_ATTRIBUTE=mail
LDAP_DISPLAY_NAME_ATTRIBUTE=uid
LDAP_USER_TO_GROUPS=true
LDAP_GROUP_ATTRIBUTE="memberOf"
LDAP_REMOVE_FROM_GROUPS=false
LDAP_AUTO_CONFIRM_EMAIL=true
LDAP_TLS_INSECURE=false
LDAP_FOLLOW_REFERRALS=true
APP_DEBUG=true
@ssddanbrown commented on GitHub (Apr 14, 2020):
Hi @kabaga
Thanks for the detailed report and apologies about this. Just tested and can confirm the problem here. Looks like some work I did in
v0.28, to better separate & align the authentication systems, has had the side-effect of producing this awkward behaviour.I'll look to update this soon.
Until then, as a work-around, You can alternatively update the "External Authentication ID" in the database directly. You can find this on the
userstable, on theexternal_auth_idcolumn. You could do this to at least gain yourself access as an LDAP-login admin so that you can then login and continue via the interface as normal.@ssddanbrown commented on GitHub (Apr 25, 2020):
Within
519283e643, I've updated the authentication behaviour so that admin users will be authenticated across all authentication types upon login. This will be part of the next BookStack patch release.@eoli3n commented on GitHub (Apr 3, 2024):
It seems that this is not working, I just created a admin user with
And when I try to login with
AUTH_METHOD=ldap, I getinformations doesn't match any account.@ssddanbrown commented on GitHub (Apr 3, 2024):
@eoli3n You'd need to change to
AUTH_METHOD=standardtemporarily to sign in via an email/password account like that.@eoli3n commented on GitHub (Apr 3, 2024):
No way to force standard auth only for a admin account ? Code-side I mean.
Any plan to fix this ? It seems to be a pretty important issue.
EDIT : when i switch to
standard, I now get "email should be a valid email address".EDIT : when i try to use my email, "already used"
Bookstack is a bit capricious about authentication
Why isn't it possible to put on list box, to choose "local" or "ldap", as we do in the config file, on the login screen ?
EDIT : Why would bookstack force email authentication when using
standardauth.Too much complications, IMO this should be a core feature, simple to manage.
Ability to:
standardmethod@ssddanbrown commented on GitHub (Apr 3, 2024):
@eoli3n You need to load the login page with
AUTH_METHOD=standardthen login with your admin account, then revert backAUTH_METHOD.Like most things, it is technically possible but would that introduce complications and additional concerns. I believe there is an open request for simultaneous main auth methods via another request.
I don't currently see an active issue here, so no plans to fix anything.
I'm not sure what this means. standard auth is email auth.
You shouldn't get any kind of
already usedwarning with standard auth login.@eoli3n commented on GitHub (Apr 3, 2024):
I successfully authenticated with the local admin account.
The
already usedis at creation the local admin account withphp artisan ....I connect with admin account everyday, when I miss some permissions.
In the current state, I need then one ldap admin account to easily connect without breaking production (when changing to
standard), and a local admin account when AD or network is down. Seems strange to me, things should be simpler.Because you chose it to be :) Why would ldap auth be username (here we have choice yes), and standard auth something else ?
It forces to set an email adress, which should be optionnal IMO.