Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support for Cosmos Ecosystem based wallet addresses in Verification Methods + verification Method Type condition rearrangements #340

Merged
merged 1 commit into from
Mar 15, 2023

Conversation

arnabghose997
Copy link
Contributor

  • Users can add their Cosmos based bech32 wallet addresses in their DID Documents. The format for blockchainAccountId is cosmos:<chain-id>:<bech-32 encoded wallet address>. Currently, the support is limited to the following chains:

    • Mainnet Chains:
      • Cosmos Hub (cosmoshub-4)
      • Osmosis (osmosis-1)
      • Akash Network (akashnet-2)
      • Persistence (core-1)
      • Crypto.org (crypto-org-chain-mainnet-1)
    • Testnet Chains
      • Cosmos Hub (theta-testnet-001)
      • Osmosis (osmo-test-4)
      • Stargaze (elgafar-1)
      • Persistence (test-core-1)
      • Hypersign Testnet (jagrat)
  • The branching of Verification Method Types is done based on the following relationships between itself and attributes such as blockchainAccountId and publicKeyMultibase

↓ VM Type / VM Attributes → publicKeyMultibase blockchainAccountId Valid/Invalid ?
Ed25519VerificationKey2020 Present Present Invalid
Absent Present Invalid
Present Absent Valid
EcdsaSecp256k1VerificationKey2019 Present Present Valid
Absent Present Invalid
Present Absent Valid
EcdsaSecp256k1RecoveryMethod2020 Present Present Invalid
Absent Present Valid
Present Absent Invalid

Test Report has been attached below
e2e_ssi_module_test_report.txt

@arnabghose997 arnabghose997 added enhancement New feature or request hid-node labels Mar 15, 2023
@arnabghose997 arnabghose997 self-assigned this Mar 15, 2023
@arnabghose997 arnabghose997 changed the base branch from develop to main March 15, 2023 12:04
@arnabghose997 arnabghose997 changed the base branch from main to develop March 15, 2023 12:04
@Vishwas1 Vishwas1 merged commit d27d001 into develop Mar 15, 2023
@arnabghose997 arnabghose997 deleted the cosmos-blockchain-addr-support branch March 21, 2023 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request hid-node
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants