-
Notifications
You must be signed in to change notification settings - Fork 803
Update ERC-7730: add optional integrity field amendment #1278
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
base: master
Are you sure you want to change the base?
Update ERC-7730: add optional integrity field amendment #1278
Conversation
File
|
|
The commit 49b8320 (as a parent of 3f759c9) contains errors. |
|
It's unclear to me "who" is the correct signer here. Once if've verified the signature to an address - how can I determine that this is the correct author? My gut feeling tells me this should be out of scope for the initial ERC-7730 |
Signature verification proves key control, not who you should trust. |
|
An alternative to defining the signature scheme in the ERC 7730 is to define the verifiable credential structure that can carry the integrity data. They have a type specifically dedicate for this (DocumentIntegrityCredential) and that way we can re-use the signature formats, signer identifier (DID spec) and verifier tooling. |
@lcastillo-ledger VCs/DIDs are useful, but I’d rather not add that stack to 7730. It increases complexity (resolvers, cryptosuites, verifier plumbing) for what should be a lightweight “is this descriptor intact and who signed it?” check. I would keep it minimal. |
Summary
integrityobject that lets wallets and tooling detect descriptor tampering without changing how descriptors are interpreted