Skip to content

[1.0.0] Resolve the namingContexts from storage. Correct SIGHUP.#152

Merged
ChadSikorra merged 6 commits into
FreeDSx:mainfrom
ChadSikorra:upgrade/misc-fixes
May 29, 2026
Merged

[1.0.0] Resolve the namingContexts from storage. Correct SIGHUP.#152
ChadSikorra merged 6 commits into
FreeDSx:mainfrom
ChadSikorra:upgrade/misc-fixes

Conversation

@ChadSikorra
Copy link
Copy Markdown
Contributor

Two fixes here:

  • The namingContexts advertised from the RootDSE / enforced by the backend should come from actual storage. Not the options. This is a holdover from when the server was literally a shell with no storage backend. This has significantly changed.
  • Writing a namingContext requires a system level operation. So it would need to be done in seed or applyChanges. It might make sense to have a writable config populated by the system or something. But currently not planning to go that route.
  • The SIGHUP should be an unimplemented to do. Just want to suppress the builtin behavior where a SIGHUP triggers a server shutdown, which it should not do.

…ever was configured in options. What is in storage for the NC will be restricted to system writes in a subsequent commit.
@ChadSikorra ChadSikorra merged commit 2de5ad8 into FreeDSx:main May 29, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant