Skip to content

Give a directory domain its own join account - #41

Merged
mastacontrola merged 1 commit into
mainfrom
feat/directorygroups-join-account
Oct 16, 2026
Merged

mastacontrola merged 1 commit into
mainfrom
feat/directorygroups-join-account

Conversation

@mastacontrola

Copy link
Copy Markdown
Member

For FOGProject/fogproject#1887. Needs FOGProject/fogproject#1890, which adds the DIRECTORY_JOIN_ACCOUNT event.

Change

  • A Directory Domain carries a join account: ddJoinUser and ddJoinPwd, added by schema steps 8 and 9.
  • The plugin answers DIRECTORY_JOIN_ACCOUNT. The domain's account beats the host's own ADUser/ADPass. A domain with no account (user and password both needed) leaves the host's own in place.
  • The FOG Client, the Agent and FOS all ask through that one event, so all three get the same account.
  • The domain page has Join Username and Join Password. A blank password keeps the stored one on edit.
  • joinPwd is in the API's always sensitive tier and is dropped from the CSV export, like bindPwd.

Test

tests/directorygroups-join-account.test.php: the account rule, the listener registration, the API strip and the export strip, and the ALTER steps. It fails with the strip removed. Suite: 20 passed.

🤖 Generated with Claude Code

A domain can now carry the account its hosts join with (FOGProject/fogproject#1887).
The plugin answers core's DIRECTORY_JOIN_ACCOUNT event, so the FOG Client,
the Agent and FOS all get the domain's account. A domain with no account
leaves the host's own ADUser/ADPass in place.

- steps 8 and 9 add ddJoinUser and ddJoinPwd to directoryDomains;
- Join Username and Join Password on the domain page; a blank password
  keeps the stored one on edit;
- joinPwd is in the API's `always` tier and is dropped from the export.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mastacontrola
mastacontrola merged commit 0315112 into main Oct 16, 2026
2 checks passed
@mastacontrola
mastacontrola deleted the feat/directorygroups-join-account branch October 16, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant