smith-members

Closes #168 (closed).

Présentation

Nouvelle palette visant à remplacer l'implémentation actuelle de la WoT Smith.

Avancement :

• events
• replace duniter-wot<Instance2> by smith-members
• integration tests
• benchmarks (écriture + lancement local)
• benchmarks (sur machine de référence)
• remove duniter-wot + certifications + membership instance parameters
• clean // TODOs

Impacts si adoptée :

1. la palette duniter-wot n'est plus instanciée (ni membership ni certification ) : réalisé dans cette MR
2. la palette membership , exploitée seulement pour la WoT principale, pourrait disparaître (voir explication ici) : non réalisé dans cette MR

Architecture

La palette s'intercale entre la palette membership et authority-members.

Son rôle est de définir qui est Smith, c'est-à-dire qui fait partie du groupe de personnes capables d'interagir avec la palette authority-members et plus spécifiquement la méthode go_online().

La palette :

• en aval :
  • implémente IsMember de authority-members pour avoir un contrôle sur go_online()
  • fournit un trait OnSmithDelete qui notifie authority-members qu'un Smith a perdu son statut et donc qu'une autorité, éventuellement online, doit être exclue (via OutgoingAuthorities)
• en amont :
  • définit un trait IsWoTMember implémenté par la palette membership qui fournit qui est membre de la WoT (car seuls les membres peuvent devenir Smith)
  • implémente OnIdtyChange de la palette identity afin de retirer le statut Smith à qui perd le statut membre de la WoT

Fonctionnement

Je suis parti de l'idée que nous souhaitions conserver une toile Smith à peu près sur les mêmes règles que l'actuelle, à savoir :

• il faut être Membre (de la WoT) pour devenir Smith
• devenir Smith se fait par certification

J'ai toutefois changé quelques éléments :

• les certifications n'expirent plus, mais sont perdues par leur bénéficiaire s'il perd le statut de Smith
• les certifications (et invitations) ne sont possibles que pour les Smith online (c-à-d au statut d'Autorité au sens de la palette authority-members)
• il n'y a plus de concept d'adhésion, remplacé par l'acception d'une invitation et le maintien du statut grâce à go_online()

Processus d'inscription

Pour devenir Smith un membre de la WoT doit :

1. Être invité par un Smith existant
2. Accepter l'invitation
3. Recevoir suffisamment de certifications

Et c'est tout. A partir du moment où l'étape 3 est atteinte, le Smith peut :

• devenir autorité : il lui faut simplement faire un go_online()
• inviter et certifier d'autres smiths

Limites

Un Smith ne peut pas rester éternellement Smith et offline. Le but d'être Smith est de mettre en place un nœud validateur.

De ce fait, un Smith expire au bout de SmithInactivityMaxDuration sessions. Dans ce cas, le Smith arbore un statut Excluded et perd toutes ses certifications.

Toutefois, les certifications valides (c-à-d celles dont le bénéficiaire est Smith) restent comptabilisées au cas où le smith exclu redevenait Smith par la suite : il ne faudrait pas qu'il puisse artificiellement gonfler son stock par entrée/sortie du cercle des smiths.

Redevenir Smith

Un Smith exclu peut redevenir Smith. Mais il doit recommencer tout le processus et recevoir des certifications en repartant de zéro.

Stock de certifications

Chaque Smith dispose d'un stock de MaxByIssuer certifications valides qu'il peut émettre.

Les certifications n'ont plus de date d'expiration : celles-ci sont données "à vie de Smith non exclu". L'idée est de récompenser le comportement de Smith actif, c'est-à-dire qui est online de façon stable (ou devient offline pendant de courtes durées).

Un Smith exclu peut redevenir Smith. Ce processus d'entrée/sortie n'influe pas sur le stock d'émission MaxByIssuer qui reste le même peu importe le nombre d'entrée/sortie de la toile Smith effectuées. Ce qui augmente ou diminue son stock de certification est uniquement lié à l'émission des certifications réalisées, ou à la perte de statut Smith des autres membres forgerons.

pallets/smith-members/src/lib.rs

@@ -116,8 +116,22 @@ pub mod pallet {
     #[pallet::event]
     #[pallet::generate_deposit(pub(super) fn deposit_event)]
     pub enum Event<T: Config> {
+        /// An identity is being inivited to become a smith
+        InvitationSent {
+            idty_index: T::IdtyIndex,
+            invited_by: T::IdtyIndex,
+        },
+        /// The invitation has been accepted
+        InvitationAccepted { idty_index: T::IdtyIndex },
+        /// Certification received
+        CertificationReceived {
+            idty_index: T::IdtyIndex,
+            issued_by: T::IdtyIndex,
+        },
         /// A smith gathered enough certifications to become an authority (can call `go_online()`).
-        SmithBecameAuthority { idty_index: T::IdtyIndex },
+        PromotedToSmith { idty_index: T::IdtyIndex },
+        /// A smith has been removed from the smiths set
+        SmithExcluded { idty_index: T::IdtyIndex },
     }
 
     #[pallet::genesis_config]
@@ -248,7 +262,7 @@ pub mod pallet {
             let who = ensure_signed(origin.clone())?;
             let issuer = T::IdtyIdOf::convert(who.clone()).ok_or(Error::<T>::UnknownIssuer)?;
             Self::check_invite_smith(issuer, receiver)?;
-            Self::do_invite_smith(receiver);
+            Self::do_invite_smith(issuer, receiver);
             Ok(().into())
         }
 
@@ -299,7 +313,7 @@ impl<T: Config> Pallet<T> {
         Ok(().into())
     }
 
-    fn do_invite_smith(receiver: T::IdtyIndex) {
+    fn do_invite_smith(issuer: T::IdtyIndex, receiver: T::IdtyIndex) {
         let new_expires_on = CurrentSession::<T>::get() + T::InactivityMaxDuration::get();
         // TODO: another way to write this?
         if Smiths::<T>::get(receiver).is_some() {
@@ -321,6 +335,10 @@ impl<T: Config> Pallet<T> {
             );
         }
         ExpiresOn::<T>::append(new_expires_on, receiver);
+        Self::deposit_event(Event::<T>::InvitationSent {
+            idty_index: receiver,
+            invited_by: issuer,
+        });
     }
 
     fn check_accept_invitation(receiver: T::IdtyIndex) -> DispatchResultWithPostInfo {
@@ -339,6 +357,9 @@ impl<T: Config> Pallet<T> {
             let maybe_smith_meta = maybe_smith_meta.as_mut().expect("status checked earlier");
             maybe_smith_meta.status = SmithStatus::Pending;
         });
+        Self::deposit_event(Event::<T>::InvitationAccepted {
+            idty_index: receiver,
+        });
         Ok(().into())
     }
 
@@ -388,8 +409,12 @@ impl<T: Config> Pallet<T> {
             // expiry postponed
             let new_expires_on = CurrentSession::<T>::get() + T::InactivityMaxDuration::get();
             maybe_smith_meta.expires_on = Some(new_expires_on);
+            Self::deposit_event(Event::<T>::CertificationReceived {
+                idty_index: receiver,
+                issued_by: issuer,
+            });
             if maybe_smith_meta.status == SmithStatus::Smith {
-                Self::deposit_event(Event::<T>::SmithBecameAuthority {
+                Self::deposit_event(Event::<T>::PromotedToSmith {
                     idty_index: receiver,
                 });
             }
@@ -433,7 +458,7 @@ impl<T: Config> Pallet<T> {
                                 smith,
                                 SmithRemovalReason::OfflineTooLong,
                             );
-                            // TODO: add event? (+ add others too)
+                            Self::deposit_event(Event::<T>::SmithExcluded { idty_index: smith });
                         }
                     }
                 }
@@ -479,9 +504,7 @@ impl<T: Config> Pallet<T> {
         //     T::OnSmithDelete::on_smith_delete(idty_index, SmithRemovalReason::Blacklisted);
         // }
     }
-}
 
-impl<T: Config> Pallet<T> {
     fn provide_is_member(idty_id: &T::IdtyIndex) -> bool {
         let Some(smith) = Smiths::<T>::get(idty_id) else {
             return false;