Skip to content
This repository was archived by the owner on Apr 16, 2026. It is now read-only.

Unfinished: support ed25519 - #74

Draft
szszszsz wants to merge 17 commits into
masterfrom
39-support-ed25519
Draft

Unfinished: support ed25519#74
szszszsz wants to merge 17 commits into
masterfrom
39-support-ed25519

Conversation

@szszszsz

Copy link
Copy Markdown
Member

This is unfinished work to support ed25519, described in #39.
It works on freshly flashed devices, however when the last stable firmware is upgraded with it, the device fails to boot.
The latter has to be solved before merging in this PR.

  1. Since this code was not touched for a longer time, it might introduce some new issues, or revive old ones. Test with care.
  2. This is the first release containing linked Rust code for the ed25519 support. This comes with new problems and tests.
  3. Likely this won't fit into the smaller Nitrokey FIDO2 128kB, unless the available Resident Keys space would be significantly decreased.

Help wanted to develop and test this code. See #39 for the details.

conorpp and others added 17 commits July 21, 2021 18:15
This will be used for upcoming support of ED25519 in the 'Solo' version.

(cherry picked from commit 27e1e78)
Not sure if this is the way to do it...

Make salty build.

(cherry picked from commit 398943d)
For now:

- libsodium(-dev) is expected to be preinstalled on build system for PC
build

(cherry picked from commit fe24b9e)
This version contains the scalar fix (see
ycrypto/salty#3).

(cherry picked from commit 3963c93)
This is a kind of promise that the underlying buffers have the correct
size. We know what we are doing. Hopefully ;-)

(cherry picked from commit 471df2a)
(cherry picked from commit ac11eff)
Added the Path variable to the Dockerfile. Thisway the arm-none-eabi-gcc can be called without the user knowing the PATH.

(cherry picked from commit c924da3)
@szszszsz szszszsz added enhancement New feature or request help wanted Extra attention is needed priority:low labels Mar 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request help wanted Extra attention is needed priority:low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants