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

Updating Credential Status #157

Merged
merged 2 commits into from
May 27, 2022

Conversation

arnabghose997
Copy link
Contributor

@arnabghose997 arnabghose997 commented May 27, 2022

Updating Credential Status happens with the help of same RPC used for Registering Credential Status on chain. Following checks are placed to ensure proper update of Credential Status:

  • New Issuance date and new Expiration date shouldn't differ from the old one
  • The time when the update happens should fall between issuance date and expiration date..
  • It is the ensured that the DID Id provided is the actual issuer of the input VC Id
  • Credential Status checks
    • Only "Live", "Revoked", "Suspended" and "Expired" statuses are allowed
    • Error on passing "Live" status for a credential ID which is already "Live"
    • Reason must be provided for "Revoked", "Suspended" and "Expired" (TODO: update for Expired needs to happen automatically)
    • Revoked and Expired can't be reverted to other there statuses

@arnabghose997 arnabghose997 self-assigned this May 27, 2022
@arnabghose997 arnabghose997 linked an issue May 27, 2022 that may be closed by this pull request
@arnabghose997 arnabghose997 merged commit 92e645b into main May 27, 2022
@arnabghose997 arnabghose997 deleted the 156-implement-update-credential-status-rpc branch May 27, 2022 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement Credential Status Update
1 participant