[identity] use ed25519 key instead of curve as map key
ClosedPublic
Actions

Authored by varun on Apr 24 2023, 1:50 PM.

Details

Reviewers

• jon
bartek

Commits

rCOMM797613b5131e: [identity] use ed25519 key instead of curve as map key

Summary

we should be using the ed25519 key as the map key in DDB to be consistent with comments in the .proto file and constants.rs. this was an oversight on my part

Test Plan

checked that the DDB table map was now keyed on the correct signing public key

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Apr 24 2023, 1:50 PM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptApr 24 2023, 1:50 PM

varun added a child revision: D7596: [identity] some nits.Apr 24 2023, 1:51 PM

varun added a parent revision: D7594: [identity] verify user access token.Apr 24 2023, 1:51 PM

Harbormaster completed remote builds in B18713: Diff 25626.Apr 24 2023, 2:09 PM

varun requested review of this revision.Apr 24 2023, 2:09 PM

• jon accepted this revision.Apr 24 2023, 3:22 PM

This revision is now accepted and ready to land.Apr 24 2023, 3:22 PM

Closed by commit rCOMM797613b5131e: [identity] use ed25519 key instead of curve as map key (authored by varun). · Explain WhyApr 24 2023, 8:34 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM797613b5131e: [identity] use ed25519 key instead of curve as map key.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

client_service.rs

6 lines

Diff 25643

View Options

services/identity/src/client_service.rs

[identity] use ed25519 key instead of curve as map keyClosedPublicActions