Revocation with legacy V1 document

fixes #208 (closed)

changed milestone to %runtime-1000

added C-runtime RN-runtime labels

added 1 commit

• a95c3d89 - identity: legacy revocation

Compare with previous version

marked this merge request as ready

added 1 commit

• c0c798e7 - identity: legacy revocation

Compare with previous version

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 g1 constant) ?
  • 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.

added 1 commit

• 1955b612 - identity: legacy revocation

Compare with previous version

added 1 commit

• 43e25302 - identity: legacy revocation

Compare with previous version

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

Compare with previous version

approved this merge request

merged

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_legacy will trigger error not all trait items implemented, missing: revoke_identity_legacy.

mentioned in commit ab102b3a

mentioned in commit 2eb05ca4

mentioned in commit 7a551143

mentioned in commit cf605c4e