mirror of
https://github.com/lldap/lldap.git
synced 2026-04-26 00:36:01 +03:00
[GH-ISSUE #733] [BUG] Migration issue with key going from 0.4.3 to 0.5 #265
Labels
No labels
backend
blocked
bug
cleanup
dependencies
docker
documentation
duplicate
enhancement
enhancement
frontend
github_actions
good first issue
help wanted
help wanted
integration
invalid
ldap
pull-request
question
rust
rust
tests
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/lldap-lldap#265
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
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 @ishanjain28 on GitHub (Nov 9, 2023).
Original GitHub issue: https://github.com/lldap/lldap/issues/733
Describe the bug
lldap version 0.4.3 ignores
key_seedfield and uses key files. lldap 0.5 does not ignorekey_seedfield. This can confuse people who had configuredkey_seedwith 0.4.3 and didn't realize it auto generated a key file any way and it was using a key file.After upgrading to 0.5,
key_seedwill actually start working and users will not be able to login to any thing(the web dashboard or ldap enabled apps) if key seed and key file contain different keys.To Reproduce
Steps to reproduce the behavior:
Expected behavior
Every thing should still be accessible after upgrading
Additional context
On startup,
lldapshould warn users about potential migration issues if config containskey_seedand there is a valid private key at the path specified inkey_fileOR there is a valid private key at$(pwd)/server_key.The latter is default location for saving private key if
key_filefield was empty/unspecified.@nitnelave commented on GitHub (Jan 22, 2024):
I fixed that in a previous PR, closing now. We have way better diagnosis for that case.