Revocation with legacy V1 document
fixes #208 (closed)
Merge request reports
Activity
changed milestone to %runtime-1000
added C-runtime RN-runtime labels
requested review from @HugoTrentesaux
- est-ce important de supporter les documents de DUBP version <10 ?
- faudrait-il rendre paramétrique le nom de la monnaie (plutôt que
g1constant) ?- autant ne pas s'embêter à créer plus de couplages pour cette fonctionnalité qu'il serait bon de ne garder que quelques années.
- faudrait-il vérifier plus précisément le format ?
- puisqu'on vérifie la signature, profiter d'un bug de parsage ne permet pas de révoquer un compte qui ne nous appartient pas. En supposant que tous les clients v1 (Cesium et duniterpy, c'est tout ?) suivent rigoureusement le même format, il ne devrait pas y avoir de problème, et tant pis pour les autres.
Je pense que prendre en charge la dernière version du document est suffisant dans une grande majorité d'un petit nombre de cas.
On peut aussi partir du principe que le nom de monnaie est g1, car c'est la seule monnaie qui sera migrée.
Le seul client largement utilisé qui incite à télécharger le document de révocation est Cesium. On peut donc se limiter à ce strict format à mon avis.
-
Effectivement, vu que le document de révocation contient plus d'informations que nécessaire, c'est plus homogène de le parser côté blockchain plutôt que de soumettre les différents champs dans des arguments de l'extrinsic en incluant les passages "inutiles" et de reconstruire le payload côté runtime. Il faut mettre à jour les métadonnées du runtime pour que les tests cucumber passent, et après ça me paraît bon.
added 3 commits
-
43e25302...0fcb69f1 - 2 commits from branch
master - ea29e4d8 - identity: legacy revocation
-
43e25302...0fcb69f1 - 2 commits from branch
mentioned in commit 43c696c2
To-do for the upgrade to 1.17.0 !312 (merged): The benchmark is missing, so the autogenerated weights for
revoke_identity_legacywill trigger errornot all trait items implemented, missing: revoke_identity_legacy.Edited by Benjamin Galloismentioned in commit ab102b3a
mentioned in commit 2eb05ca4
mentioned in commit 7a551143
mentioned in commit cf605c4e
