Skip to content

client/setec: add a Keyring type#161

Merged
creachadair merged 1 commit intomainfrom
mjf/keyring-nova
Mar 10, 2026
Merged

client/setec: add a Keyring type#161
creachadair merged 1 commit intomainfrom
mjf/keyring-nova

Conversation

@creachadair
Copy link
Copy Markdown
Member

@creachadair creachadair commented Mar 5, 2026

Introduce a new Keyring type to the client library, representing a collection
of all the available versions of a secret. The Keyring allows the caller to get
any of the versions known explicitly (as Client.GetVersion) without an
additional fetch from the service, or to get the ID and value of the "active"
version explicitly.

A Keyring also supports an Update operation which polls the server for changes,
allowing the caller to observe additional versions and changes to the active
version ID.

  • Add a new method GetKeyring to the Client to fetch a keyring.
  • Update tests and documentation.

This commit does not integrate the Keyring type into the Store.
Doing so will be the task of a future change.

Introduce a new Keyring type to the client library, representing a collection
of all the available versions of a secret. The Keyring allows the caller to get
any of the versions known explicitly (as Client.GetVersion) without an
additional fetch from the service, or to get the ID and value of the "active"
version explicitly.

A Keyring also supports an Update operation which polls the server for changes,
allowing the caller to observe additional versions and changes to the active
version ID.

- Add a new method GetKeyring to the Client to fetch a keyring.
- Update tests and documentation.
@creachadair creachadair requested a review from sfarqu March 10, 2026 15:54
@creachadair creachadair marked this pull request as ready for review March 10, 2026 15:54
Copy link
Copy Markdown

@sfarqu sfarqu left a comment

Choose a reason for hiding this comment

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

Looks good, as per prior discussions about requirements for corp/37369.

@creachadair creachadair merged commit dcd97e4 into main Mar 10, 2026
1 check passed
@creachadair creachadair deleted the mjf/keyring-nova branch March 10, 2026 22:14
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