Skip to content

[BB-690] Baton ID support for matching AD users and groups#33

Merged
johnallers merged 16 commits into
mainfrom
jallers/fix-groups
Aug 28, 2025
Merged

[BB-690] Baton ID support for matching AD users and groups#33
johnallers merged 16 commits into
mainfrom
jallers/fix-groups

Conversation

@johnallers

@johnallers johnallers commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

Supports Active Directory user and group matching using the baton-active-directory connector as the shared identity source (external resource). Users and groups are matched, by name, to the "downlevel_logon_name" profile attribute value in the Active Directory connector's resource.

This feature is dependent upon https://github.com/ConductorOne/baton-active-directory/pull/72.

@johnallers johnallers changed the title Baton ID support for matching AD users and groups [BB-690] Baton ID support for matching AD users and groups Jul 30, 2025

@ggreer ggreer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why this is a draft. The changes make sense to me.

Comment thread pkg/connector/group.go
var ret []*v2.Resource
for _, principalModel := range principals {
r, err := resource.NewUserResource(
r, err := resource.NewGroupResource(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😆

@johnallers johnallers marked this pull request as ready for review August 28, 2025 12:39
@johnallers johnallers merged commit 11a4664 into main Aug 28, 2025
7 checks passed
@johnallers johnallers deleted the jallers/fix-groups branch August 28, 2025 12:50
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.

2 participants