fix svcLdapAdmin seed: simpleSecurityObject is what actually grants userPassword #117

Merged
lament merged 1 commit from fix-svcldap-seed into main 2026-07-27 17:50:58 -05:00
Owner

the seed's ldapadd for cn=svcLdapAdmin used objectClass: organizationalRole + krbPrincipalAux only - neither grants userPassword, so the follow-up ldapmodify hit Object class violation (65): attribute 'userPassword' not allowed and silently no-opped (the seed script swallows failures with || true). caught by a live verification against verdandi: the entry existed but had no userPassword/krbPrincipalName set at all.

  • adds simpleSecurityObject as a second auxiliary class (same fix svc-ldap already needed for the same reason)
  • since that class MUSTs userPassword at creation, the {SASL} pointer moves into the initial ldapadd instead of relying on the modify step to set it for the first time
  • the modify block still re-asserts userPassword + sets krbPrincipalName, unchanged
the seed's ldapadd for cn=svcLdapAdmin used objectClass: organizationalRole + krbPrincipalAux only - neither grants userPassword, so the follow-up ldapmodify hit `Object class violation (65): attribute 'userPassword' not allowed` and silently no-opped (the seed script swallows failures with `|| true`). caught by a live verification against verdandi: the entry existed but had no userPassword/krbPrincipalName set at all. - adds `simpleSecurityObject` as a second auxiliary class (same fix svc-ldap already needed for the same reason) - since that class MUSTs userPassword at creation, the {SASL} pointer moves into the initial ldapadd instead of relying on the modify step to set it for the first time - the modify block still re-asserts userPassword + sets krbPrincipalName, unchanged
fix svcLdapAdmin seed: simpleSecurityObject is what actually grants userPassword
All checks were successful
CI / check (pull_request) Successful in 54s
CI / deploy (pull_request) Has been skipped
CI / build (pull_request) Successful in 2m8s
CI / dry-activate (pull_request) Successful in 32s
CI / check (push) Has been skipped
CI / build (push) Has been skipped
CI / dry-activate (push) Has been skipped
CI / deploy (push) Successful in 2m39s
24336e0842
lament scheduled this pull request to auto merge when all checks succeed 2026-07-27 17:47:43 -05:00
lament merged commit 24336e0842 into main 2026-07-27 17:50:58 -05:00
lament deleted branch fix-svcldap-seed 2026-07-27 17:50:58 -05:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
lament/pantheon!117
No description provided.