Skip to content

CA-428461: make trusted-domain attribute enrichment best-effort#7126

Open
liulinC wants to merge 1 commit into
xapi-project:feature/ldapsfrom
liulinC:private/linl/CA-428461
Open

CA-428461: make trusted-domain attribute enrichment best-effort#7126
liulinC wants to merge 1 commit into
xapi-project:feature/ldapsfrom
liulinC:private/linl/CA-428461

Conversation

@liulinC

@liulinC liulinC commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Fold Wbinfo.kdc_of_domain (wbinfo --getdcname) into the same match as Ldap.query_user in query_subject_information_user, so a DC-location or LDAP query failure falls back to default_account instead of aborting subject creation via let*. Fixes subject-add for one-way trusted-domain users when getdcname cannot locate the trusted DC; attributes are refreshed later by update_all_subjects.

@liulinC liulinC force-pushed the private/linl/CA-428461 branch 2 times, most recently from de76c98 to 900b816 Compare June 12, 2026 04:40
Fold Wbinfo.kdc_of_domain (wbinfo --getdcname) into the same match as
Ldap.query_user in query_subject_information_user, so a DC-location or
LDAP query failure falls back to default_account instead of aborting
subject creation via let*. Fixes subject-add for one-way trusted-domain
users when getdcname cannot locate the trusted DC; attributes are
refreshed later by update_all_subjects.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Lin Liu <Lin.Liu01@cloud.com>
@liulinC liulinC force-pushed the private/linl/CA-428461 branch from 900b816 to d57c84d Compare June 12, 2026 06:35
Comment thread ocaml/xapi/extauth_plugin_ADwinbind.ml
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.

3 participants