[1.0.0] Add an extensible storage backend system for the LDAP server.#96
Merged
ChadSikorra merged 35 commits intoFreeDSx:1.0from Apr 11, 2026
Merged
[1.0.0] Add an extensible storage backend system for the LDAP server.#96ChadSikorra merged 35 commits intoFreeDSx:1.0from
ChadSikorra merged 35 commits intoFreeDSx:1.0from
Conversation
6e436f5 to
b5d9773
Compare
a678007 to
615c047
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## 1.0 #96 +/- ##
============================================
- Coverage 88.12% 86.29% -1.83%
- Complexity 2290 2600 +310
============================================
Files 174 196 +22
Lines 7225 8173 +948
============================================
+ Hits 6367 7053 +686
- Misses 858 1120 +262 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
e1d9a2a to
3c513e5
Compare
Fix tests. Fix where coroutine hooks are enabled.
3c513e5 to
172343b
Compare
32f8ab7 to
f87253b
Compare
c6ff998 to
8a08723
Compare
…y are doing (as there are quite a few of them now).
…er pcntl or swoole.
…nto at resolution time to make custom ones less awkward.
…figuration way less awkward.
4506f5d to
df6003e
Compare
38d1b7d to
b025360
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
First pass at an extensible storage backend system for the LDAP server. Supports simple file based / in-memory by default. Adds a Swoole based server as an optional dependency, so pcntl isn't the only way to run it. It's also needed to support an in-memory approach. Though the file approach should work for either.
I have purposefully avoided some kind of MySQL / other explicitly DB backed solution for now. I will create some docs with some major caveats. This should really only be used for development purposes. The server component was not meant for high load or anything substantial. It's quite limited in its LDAP capabilities, though it does support quite a lot now.
I also need to go through this quite a bit before it's ready.
Edit: This turned into a beast. I really had to refactor and refine a ton of things to get a pluggable backend working. So this is almost a complete reworking of the old way you'd extend the backend server component. But it's way more customizable now and things are split apart in a more sane way: