ChainAgnostic / CASA

Chain Agnostic Standards Alliance organizational homebase
https://alliance.chainagnostic.org
114 stars 41 forks source link

Editorial WG - 29 June 2023 #86

Open bumblefudge opened 1 year ago

bumblefudge commented 1 year ago

CASA Editorial 29 June

PRs to refine/move to close

Ongoing projects/topics

Backlog

haardikk21 commented 1 year ago

FWIW my Tezos SIWX implementation is inspired from the one at Ceramic, though is less restrictive. Ceramic's implementation only allows for Tezos Ed25519 keys - whereas ours allows for all types of Tezos keys as long as they can be validated using Taquito since a lot of popular Tezos wallets don't create Ed25519 keys by default.

This part will definitely fail during cross-testing.

bumblefudge commented 1 year ago

@haardikk21 @zachferland

Ceramic's implementation only allows for Tezos Ed25519 keys - whereas ours allows for all types of Tezos keys as long as they can be validated using Taquito since a lot of popular Tezos wallets don't create Ed25519 keys by default.

I went to look up what the tezos 122 profile says about non-Ed25519 keys... and saw that the relevant PR was never merged because no one ever confirmed Qibinglee's work on the other key/address types 🤦 . Maybe if you've already implemented you can approve that PR and we can go from there?

My assumption (perhaps overhasty) was that since tz1, tz2 and tz3 keys are separate entries/regexs/etc in did:pkh, they'd be separate classes or subclasses of account eligible for a CAIP-122 flow/interaction. As such, they should be tested (and cross-tested) separately anyways-- i.e., an implementation might still be considered valid using a subset of the available tezos keytypes, particularly since not all wallets expose all 3 and they could be deprecated on different timelines over the years...

But my assumption and Qibing's profile don't quite match so if someone cares they should probably argue their case, suggest changes, etc :D