Page MenuHomePhabricator
Differential D8298

[identity] fix format of ed25519 public key and signature
ClosedPublic
Actions

Authored by varun on Jun 22 2023, 11:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 29, 11:49 PM
Unknown Object (File)
Sat, Nov 23, 2:02 PM
Unknown Object (File)
Sat, Nov 23, 2:02 PM
Unknown Object (File)
Sat, Nov 23, 2:02 PM
Unknown Object (File)
Sat, Nov 23, 2:02 PM
Unknown Object (File)
Sat, Nov 9, 6:59 AM
Unknown Object (File)
Thu, Nov 7, 4:30 PM
Unknown Object (File)
Oct 12 2024, 11:00 AM
View All Files
Subscribers
ashoat
tomek

Details

Reviewers
• jon
bartek
Commits
rCOMMb97e7a853b67: [identity] fix format of ed25519 public key and signature
Summary

the public key and signature should be decoded to byte slices.

Test Plan

tested two diffs later in stack when we verify signatures with the public key

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun created this revision.Jun 22 2023, 11:43 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 22 2023, 11:43 PM
varun added a child revision: D8299: [keyserver] improve error messages in rust-node-addon.Jun 22 2023, 11:45 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Jun 23 2023, 12:08 AM
Harbormaster failed remote builds in B20444: Diff 28026!
varun requested review of this revision.Jun 23 2023, 1:23 PM
• jon added a comment.Jun 26 2023, 5:55 AM

LGTM, would like for bartek to take a look.

bartek accepted this revision.Jun 26 2023, 3:21 PM
This revision is now accepted and ready to land.Jun 26 2023, 3:21 PM
This revision was landed with ongoing or failed builds.Jun 26 2023, 6:54 PM
Closed by commit rCOMMb97e7a853b67: [identity] fix format of ed25519 public key and signature (authored by varun). · Explain Why
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMMb97e7a853b67: [identity] fix format of ed25519 public key and signature.

Revision Contents
Changeset List

PathSize
services/
identity/
1 line
src/
19 lines

Diff 28127

View Options

services/identity/Cargo.lock

Loading...
View Options

services/identity/Cargo.toml

Loading...
View Options

services/identity/src/reserved_users.rs

Loading...