Skip to content

Add support for FISO2/U2F sk- key types - #56

Closed
nicolas-graves wants to merge 1 commit into
artyom-poptsov:masterfrom
nicolas-graves:sk-keys
Closed

Add support for FISO2/U2F sk- key types#56
nicolas-graves wants to merge 1 commit into
artyom-poptsov:masterfrom
nicolas-graves:sk-keys

Conversation

@nicolas-graves

Copy link
Copy Markdown
Collaborator

I noticed a few key types were missing, it's probably a good idea to add them.

  • libguile-ssh/key-type.c (key_types): Add missing key types sk-ecdsa, sk-ecdsa-cert01, sk-ed25519, sk-ed25519-cert01.
  • tests/keys/skecdsakey.pub: Add new test key.
  • tests/keys/sked25519key.pub: Likewise.
  • tests/Makefile.am (EXTRA_DIST): Record them.
  • tests/common.scm (%skecdsakey-pub, %sked25519key-pub): Add them.
  • tests/key.scm (unless-sk-supported): Add helper macro. (%skecdsakey-pub-string, %sked25519-pub-string): Add helper variables. ("public-key-from-file: SK-ECDSA", "public-key-from-file: SK-ED25519") ("get-key-type: SK-ECDSA", "get-key-type: SK-ED25519") ("string->public-key, SK-ECDSA", "string->public-key, SK-ED25519"): Add tests.
  • NEWS: Add a entry.

@nicolas-graves

Copy link
Copy Markdown
Collaborator Author

Disclaimer: I partly used gen AI to generate tests

I noticed a few key types were missing, it's probably a good idea to add
them.

* libguile-ssh/key-type.c (key_types): Add missing key types sk-ecdsa,
sk-ecdsa-cert01, sk-ed25519, sk-ed25519-cert01.
* tests/keys/skecdsakey.pub: Add new test key.
* tests/keys/sked25519key.pub: Likewise.
* tests/Makefile.am (EXTRA_DIST): Record them.
* tests/common.scm (%skecdsakey-pub, %sked25519key-pub): Add them.
* tests/key.scm (unless-sk-supported): Add helper macro.
(%skecdsakey-pub-string, %sked25519-pub-string): Add helper variables.
("public-key-from-file: SK-ECDSA", "public-key-from-file: SK-ED25519")
("get-key-type: SK-ECDSA", "get-key-type: SK-ED25519")
("string->public-key, SK-ECDSA", "string->public-key, SK-ED25519"): Add tests.
* NEWS: Add a entry.
@nicolas-graves nicolas-graves changed the title Add support for FISO2/U2F sk- key types. Add support for FISO2/U2F sk- key types Aug 11, 2026
@artyom-poptsov

Copy link
Copy Markdown
Owner

Merged to the master branch as 2e6c46e

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