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

Make more APIs public #140

Merged
merged 4 commits into from
Dec 1, 2023
Merged

Conversation

sosthene-nitrokey
Copy link
Contributor

These APIs improve debugging or are required for custom backend implementations.

Built on top of #139 that will need to be merged first.

  • Add debug implementations to BackendId and make it Clone. This allows better debugging.
  • Add methods to read the u128 backing a KeyId and reading it. This can already be done through the serde implementation. This makes it possible to a custom backend to encode informations so that it can differentiate between keys from the core backend and keys it generated itself.

@sosthene-nitrokey sosthene-nitrokey mentioned this pull request Nov 13, 2023
@sosthene-nitrokey sosthene-nitrokey marked this pull request as ready for review November 22, 2023 10:15
 This can be useful for backends which can use it to encode additional information inside of the KeyId itself (128 bits is a lot)

 This is already possible to acheive through the serde implementation, so this doesn't really add any unavailable functionality.
@sosthene-nitrokey sosthene-nitrokey merged commit 91c59a7 into trussed-dev:main Dec 1, 2023
2 checks passed
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