Skip to content

feat(scimdirectory): add credentials method for pull-provider directories - #579

Merged
gabrielmeloc22 merged 3 commits into
v3from
gabriel/orgs-1480-google-directory-api-adapter-full-sync-dapi
Jul 23, 2026
Merged

feat(scimdirectory): add credentials method for pull-provider directories#579
gabrielmeloc22 merged 3 commits into
v3from
gabriel/orgs-1480-google-directory-api-adapter-full-sync-dapi

Conversation

@gabrielmeloc22

Copy link
Copy Markdown
Contributor

Summary

Adds a Credentials method to the scimdirectory client for uploading pull-provider credentials (e.g. a Google service-account key + subject email) to a directory. Backs the BAPI credentials endpoint used to enable pull-based directories.

Changes

  • Add CredentialsParams{ServiceAccountJSON, SubjectEmail} and Client.Credentials(ctx, id, params)POST /scim_directories/{id}/credentials, returning the updated SCIMDirectory.
  • Regenerate the package-level Credentials wrapper in api.go.
  • Add TestCredentials covering the request path and response mapping.

@gabrielmeloc22
gabrielmeloc22 requested a review from a team as a code owner July 16, 2026 14:27

@NicolasLopes7 NicolasLopes7 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.

lgtm

Comment thread scimdirectory/client.go Outdated
gabrielmeloc22 and others added 2 commits July 20, 2026 11:14
Co-authored-by: nicolas lopes <57234795+NicolasLopes7@users.noreply.github.com>
…ename

The e0a3c51 rename of Client.Credentials -> AddCredentials left the generated
package-level wrapper in api.go and the client test calling the old method,
which broke the build (and lint/test in CI). Regenerate api.go and update the
test to the new name.
@gabrielmeloc22
gabrielmeloc22 merged commit 1d2d327 into v3 Jul 23, 2026
7 checks passed
@gabrielmeloc22
gabrielmeloc22 deleted the gabriel/orgs-1480-google-directory-api-adapter-full-sync-dapi branch July 23, 2026 23: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