feat: Identity removal reason
All threads resolved!
All threads resolved!
Fix #106 (closed)
Merge request reports
Activity
Filter activity
added RN-runtime label
changed milestone to %runtime-700
- Resolved by Hugo Trentesaux
Si je résume on a :
enum IdtyRemovalReason { Expired, // une identité expire si elle n'a pas été renouvelée à temps Manual, Revoked, // uniquement par certificat de révocation Other(Other), }
et
enum IdtyRemovalWotReason { LostCert, // member passé sous le seuil de certifications dans la toile principale MembershipExpired, // adhésion non renouvelée Other, }
-
Manual
n'est pas utilisé (et devrait l'être dans certains tests) -
Other(Other)
ne correspond dans notre cas qu'à des actions de type web of trust, on pourrait l'appelerWot(Reason)
, de toutes façons les pallets sont couplées, ou mêmeExternal(Reason)
-
Other
je ne comprends pas dans quel cas le retrait de l'identité est dû à la toile de confiance mais ce n'est ni une certification perdue, ni une expiration d'adhésion
D'ailleurs en relisant ça, je me rends compte que la logique de retrait des identités en cas de perte de certification n'est toujours pas claire (même si c'est moi qui l'ai fait dans !166 (merged)). Il faut que je revoie ça et ajoute des tests.
-
- Resolved by Pascal Engélibert
added 5 commits
-
8ae73e0f...b575a32a - 4 commits from branch
master
- 0cce5387 - feat: Identity removal reason
-
8ae73e0f...b575a32a - 4 commits from branch
requested review from @HugoTrentesaux
added 2 commits
enabled an automatic merge when the pipeline for cadb0753 succeeds
mentioned in commit 71bc932e
changed milestone to %runtime-600
changed milestone to %runtime-700
Please register or sign in to reply