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

Pre-TestnetV2 Refactor #374

Merged
merged 8 commits into from
Nov 9, 2023
Merged

Pre-TestnetV2 Refactor #374

merged 8 commits into from
Nov 9, 2023

Conversation

arnabghose997
Copy link
Contributor

  • SSI proto package name has been renamed from hypersignprotocol.hidnode.ssi to hypersign.ssi.v1
  • Client Spec has been reduced to an enum type. ADR036Signer address is inferred from the blockchainAccountId
  • A common Proof message is introduced for DID, Credential Schema and Credential Status documents.
    message DocumentProof {
        string type = 1;
        string created = 2;
        string verificationMethod = 3;
        string proofPurpose = 4;
        string proofValue = 5;
        ClientSpecType clientSpecType = 6; 
    }
    It also replaces the existing Schema Proof, Credential Status Proof and DID SignInfo messages. Proof message is only stored on-chain for Credential Schema and Credential Status
  • Credential Status message has been revamped by eliminating statuses such as Live and Expired. Only Revocation and Suspension is tracked. Claim message has been discarded, making fields such as id, remarks part of the main Credential Status message. The attribute credentialHash has been renamed to merkleRootHash to indicate that the Hash represents the Merkle Root Hash of the VC
  • CreateDID RPC has been renamed to RegisterDID. Updates of any CredentialStatus document will done through a seperate RPC: UpdateCredentialStatus. Updates of any CredentialStatus document will done through a seperate RPC: UpdateCredentialSchema.
  • json_name proto tag has been added to context attribute of DidDocument to support @context field in generated TS client
  • protocgen-ts.sh script has been added to generate TS client files. This approach provides more flexibility in terms of generation options compared to using ignite CLI tool

…ce with EdDsaCryptoSuite2020 W3C specification; this includes canonizing the document using RDF dataset normalization and expecting publicKeyMutibase of verification method type Ed25519VerificationKey2020 to carry a 2-byte prefix as described in the said specification
…LD struct. This issue was solved by converting the jsonLd struct to an interface{} object and fed to the normalize function
	- SSI proto package name has been renamed from hypersignprotocol.hidnode.ssi to hypersign.ssi.v1

	- Client Spec has been reduced to an enum type. ADR036Signer address is infered from the blockchainAccountId

	- A common Proof message is introduced for DID, Credential Schema and Credential Status documents. It also replaces the existing Schema Proof, Credential Status Proof and DID SignInfo messages. Proof message is only stored on-chain for Credential Schema and Credential Status

	- Credential Status message has been revamped by eliminating statuses such as Live and Expired. Only Revocation and Suspension is tracked. Claim message has been discarded, making fields such as id, remarks part of the main Credential Status message. The attribute credentialHash has been renamed to merkleRootHash to indicate that the Hash represents the Merkle Root Hash of the VC

	- CreateDID RPC has been renamed to RegisterDID. Updates of any CredentialStatus document will done through a seperate RPC: UpdateCredentialStatus. Updates of any CredentialStatus document will done through a seperate RPC: UpdateCredentialSchema.

	- json_name proto tag has been added to context attribute of DidDocument to support @context field in generated TS client

	- protocgen-ts.sh script has been added to generate TS client files. This approach provides more flexibility in terms of generation options compared to using ignite CLI tool
@arnabghose997 arnabghose997 self-assigned this Oct 19, 2023
@arnabghose997 arnabghose997 changed the title Refactor Pre-TestnetV2 Refactor Nov 1, 2023
@Vishwas1 Vishwas1 marked this pull request as ready for review November 8, 2023 05:25
@arnabghose997 arnabghose997 merged commit c0928a1 into develop Nov 9, 2023
2 checks passed
@arnabghose997 arnabghose997 deleted the pre-testnet-factor branch November 9, 2023 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants