Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Organization support for client credentials #733

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kailash-b
Copy link
Contributor

@kailash-b kailash-b commented Oct 4, 2024

Changes

Please describe both what is changing and why this is important. Include:

  • Support for Default Organisation in clients

    1. Added new field default_organization to GET/PATCH endpoints /api/v2/clients/{id}
    2. Added new field default_organization to POST endpoints /api/v2/clients/
  • Support query param q for GET /api/v2/clients

    1. q=client_grant.organization_id:org_sampleorgid213
    2. q=client_grant.allow_any_organization:true
  • Support query param grant_ids for GET /api/v2/organizations/{org_id}/client-grants

  • Support query param identifiers for GET /api/v2/resource-servers

References

Testing

Added test cases for testing above end-points.
Tested the implementation manually according to this internal document

  • This change adds unit test coverage

  • This change adds integration test coverage

  • This change has been tested on the latest version of the platform/language or why not

Checklist

@kailash-b kailash-b force-pushed the feature/org-support-client-cred branch from 6563cef to 678c858 Compare October 4, 2024 08:15
@kailash-b kailash-b force-pushed the feature/org-support-client-cred branch from 678c858 to 09dcab5 Compare October 4, 2024 08:24
@kailash-b kailash-b marked this pull request as ready for review October 4, 2024 09:30
@kailash-b kailash-b requested a review from a team as a code owner October 4, 2024 09:30
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