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

Governance metadata is not validated #906

Open
saratomaz opened this issue Sep 20, 2024 · 2 comments
Open

Governance metadata is not validated #906

saratomaz opened this issue Sep 20, 2024 · 2 comments
Assignees
Labels
conway-feature intra-era-hardfork feature to be added for protocol version 10

Comments

@saratomaz
Copy link

Description

There is no metadata validation for the governance metadata. Right now is possible to submit a governance action with wrong metadata/missing required fields and the only time we can see an error is when dbsync try to catch the data.

Steps to Reproduce

e.g.

  1. run cardano-cli conway governance drep metadata-hash --drep-metadata-file drep_metadata.json
    drep_metadata.json -> missing givenName that is marked as Compulsory in cip-0119
  2. No error is returned

Additional Context

CIPs related to governance metadata:
https://github.com/cardano-foundation/CIPs/tree/master/CIP-0100
https://github.com/cardano-foundation/CIPs/tree/master/CIP-0108
https://github.com/cardano-foundation/CIPs/tree/master/CIP-0119

@kderme
Copy link
Contributor

kderme commented Sep 20, 2024

In case cli plans to support this, we have implemented metadata parsers following the above cips https://github.com/IntersectMBO/cardano-db-sync/blob/master/cardano-db-sync/src/Cardano/DbSync/OffChain/Vote/Types.hs

@palas palas self-assigned this Sep 23, 2024
@CarlosLopezDeLara CarlosLopezDeLara added conway-feature intra-era-hardfork feature to be added for protocol version 10 and removed conway-era-integration-bug labels Sep 24, 2024
@gitmachtl
Copy link
Contributor

cardano-signer is doing checks if a metadata file is valid, also checks about the signatures.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conway-feature intra-era-hardfork feature to be added for protocol version 10
Projects
None yet
Development

No branches or pull requests

5 participants