Update LDAP_USER_FILTER env option to use new placeholder format #4683

New Issue

OVERLORD · 2026-02-05T09:10:47+03:00

OVERLORD commented

2026-02-05 09:10:47 +03:00

Originally created by @ssddanbrown on GitHub (Apr 22, 2024).

Update LDAP_USER_FILTER env option to use {user} in addition to ${user}.
The existing ${user} technically can be considered as a nested variable by dotenv. Most cases it's fine unless a user variable is defined but best to move away from this.
Doing something similar for #4732, so best to align these now.
Keep support for old format for forwards compatibility though.
Don't think it will conflict with existing cases, but maybe should still alert about this?

Doc Updates

Update existing ldap docs with new format.
Update advisory to suggest change and potential compatibility?

Originally created by @ssddanbrown on GitHub (Apr 22, 2024). Update `LDAP_USER_FILTER` env option to use `{user}` in addition to `${user}`. The existing `${user}` technically can be considered as a nested variable by dotenv. Most cases it's fine unless a `user` variable is defined but best to move away from this. Doing something similar for #4732, so best to align these now. Keep support for old format for forwards compatibility though. Don't think it will conflict with existing cases, but maybe should still alert about this? ### Doc Updates - Update existing ldap docs with new format. - Update advisory to suggest change and potential compatibility?

OVERLORD added the 🔧 Maintenance 📖 Docs Update 🏭 Back-End labels 2026-02-05 09:10:47 +03:00

OVERLORD closed this issue

2026-02-05 09:10:49 +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#4683

Update LDAP_USER_FILTER env option to use new placeholder format #4683

Doc Updates

Update `LDAP_USER_FILTER` env option to use new placeholder format #4683