Skip to content

Conversation

@tanya732
Copy link
Contributor

@tanya732 tanya732 commented May 30, 2025

Changes

Added 2 New Endpoints

Path HTTP Method Method Name
/connection/{id}/keys GET getKeys
/connections/{id}/keys/rotate POST rotateKey

Added methods to:
Retrieve connection keys: getKeys(String connectionId)
Rotate connection keys: rotateKey(String connectionId)

New Model Classes:
ConnectionKeys: Represents a key associated with a connection, including fields for key ID, algorithm, certificate, fingerprint, status, and timestamps.
RotateKey: Represents the response from a key rotation, including the key ID and certificate.

Tests:
Added and updated unit tests to cover the new key management methods and models.
New test resources for simulating API responses for connection keys and key rotation.

Manual Testing Code

Get domain and apiToken from tenant
ManagementAPI mgmt = ManagementAPI.newBuilder("{YOUR_DOMAIN}", "{YOUR_API_TOKEN}").build();

List<ConnectionKeys> keys = api.connections().getKeys("<ID>").execute().getBody();
api.connections().rotateKey("<ID>").execute();

Testing

Please describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

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

Checklist

@tanya732 tanya732 marked this pull request as ready for review June 9, 2025 11:36
@tanya732 tanya732 requested a review from a team as a code owner June 9, 2025 11:36
@tanya732 tanya732 merged commit 215d59f into master Jun 20, 2025
6 checks passed
@tanya732 tanya732 deleted the sdk-5976-client-assertion-jwt-support branch June 20, 2025 04:36
@tanya732 tanya732 mentioned this pull request Jun 20, 2025
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