# Comportement bizarre de Duniter 1.6.11

**URL:** https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592
**Category:** Duniter-v1
**Created:** [9 November 2017 19:20 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592 "2017-11-09T19:20:16Z")
**Posts on this page:** 13
**Page:** 1

<div class="post-metadata">

### Author: ![gerard94](https://forum.duniter.org/user_avatar/forum.duniter.org/gerard94/32/58_2.png) [@gerard94](https://forum.duniter.org/u/gerard94)
#### Post date: [9 November 2017 19:20 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/1 "2017-11-09T19:20:16Z")

</div>

Depuis ce matin, régulièrement, mon noeud veut admettre comme membre HydenStark1 (pubkey : GFZPHnrwNMoJZiYaUPi8S7qhHxbkJ3TVpHapVRzLLQgj), dont l’identité a déjà été révoquée. Donc il forke, puis rentre dans le rang pour recommencer au bloc suivant.

---

<div class="post-metadata">

### Author: ![cgeek](https://forum.duniter.org/user_avatar/forum.duniter.org/cgeek/32/279_2.png) [@cgeek](https://forum.duniter.org/u/cgeek)
#### Post date: [9 November 2017 19:23 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/2 "2017-11-09T19:23:33Z")

</div>

À noter que ce bug ne provient probablement pas de la v1.6.11, mais plutôt que les événements se superposent. [Ticket#1190](https://github.com/duniter/duniter/issues/1190) créé.

---

<div class="post-metadata">

### Author: ![gerard94](https://forum.duniter.org/user_avatar/forum.duniter.org/gerard94/32/58_2.png) [@gerard94](https://forum.duniter.org/u/gerard94)
#### Post date: [9 November 2017 21:04 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/3 "2017-11-09T21:04:58Z")

</div>

🤔

---

<div class="post-metadata">

### Author: ![elois](https://forum.duniter.org/user_avatar/forum.duniter.org/elois/32/1541_2.png) [@elois](https://forum.duniter.org/u/elois)
#### Post date: [9 November 2017 21:30 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/4 "2017-11-09T21:30:09Z")

</div>

> [@gerard94](#):
>
> Depuis ce matin, régulièrement, mon noeud veut admettre comme membre HydenStark1 (pubkey : GFZPHnrwNMoJZiYaUPi8S7qhHxbkJ3TVpHapVRzLLQgj), dont l’identité a déjà été révoquée. Donc il forke, puis rentre dans le rang pour recommencer au bloc suivant.

Comment le vois tu ? dans les log ? Si oui peut tu les partager ?

---

<div class="post-metadata">

### Author: ![gerard94](https://forum.duniter.org/user_avatar/forum.duniter.org/gerard94/32/58_2.png) [@gerard94](https://forum.duniter.org/u/gerard94)
#### Post date: [9 November 2017 22:10 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/5 "2017-11-09T22:10:15Z")

</div>

Je le vois directement dans la table block de la bdd.

 ![Capture d'écran de 2017-11-09 23-07-11](https://forum.duniter.org/uploads/default/original/2X/6/607ce9dbc3a3ae6c57c6ed44fee5e0ce8d5fd4f1.png)

---

<div class="post-metadata">

### Author: ![elois](https://forum.duniter.org/user_avatar/forum.duniter.org/elois/32/1541_2.png) [@elois](https://forum.duniter.org/u/elois)
#### Post date: [9 November 2017 22:43 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/6 "2017-11-09T22:43:14Z")

</div>

@gerard94 merci et merci à @cgeek aussi pour le titre du ticket qui m’a mis sur la voie 🙂

Il y a effectivement un bug semi-critique dans Duniter, et qui n’a rien a voir avec la 1.6, il est présent peut être depuis toujours, au moins depuis la dernière réécriture du module prover.

Et ce bug est bien lié au problème anormal de refus de mes blocs par le réseau.

Explications :  
Lorsque duniter prépare le contenu du prochain bloc a écrire (c’est le module prover qui fait çà), il n’exclu pas les clés révoqués, ainsi une clé révoquée mais qui redemande a être membre va être réinscrite dans le bloc, et donc refusée par le réseau…

Je viens de coder un correctif mais je n’ai pas encore pu le tester car ma clé membre est exclu du calcul des blocs, je doit attendre …

---

<div class="post-metadata">

### Author: ![gerard94](https://forum.duniter.org/user_avatar/forum.duniter.org/gerard94/32/58_2.png) [@gerard94](https://forum.duniter.org/u/gerard94)
#### Post date: [9 November 2017 23:06 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/7 "2017-11-09T23:06:19Z")

</div>

Merci à toi !

---

<div class="post-metadata">

### Author: ![elois](https://forum.duniter.org/user_avatar/forum.duniter.org/elois/32/1541_2.png) [@elois](https://forum.duniter.org/u/elois)
#### Post date: [9 November 2017 23:20 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/8 "2017-11-09T23:20:10Z")

</div>

Youhou mon correctif fonctionne et passe tout les tests 🙂

[https://github.com/duniter/duniter/commit/a86503a9eccb087e23a24ff662b7dfe2b2f399f6](https://github.com/duniter/duniter/commit/a86503a9eccb087e23a24ff662b7dfe2b2f399f6)

---

<div class="post-metadata">

### Author: ![Inso](https://forum.duniter.org/user_avatar/forum.duniter.org/inso/32/1229_2.png) [@Inso](https://forum.duniter.org/u/Inso)
#### Post date: [10 November 2017 04:35 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/9 "2017-11-10T04:35:04Z")

</div>

Il mériterait l’écriture de son propre test aussi non ?

---

<div class="post-metadata">

### Author: ![cgeek](https://forum.duniter.org/user_avatar/forum.duniter.org/cgeek/32/279_2.png) [@cgeek](https://forum.duniter.org/u/cgeek)
#### Post date: [10 November 2017 07:02 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/10 "2017-11-10T07:02:58Z")

</div>

D’ailleurs ça ne se voit pas ici, mais ces blocs sont marqués comme « fork », ils sont présents dans la BDD comme blocs potentiels (générés par ton propre nœud) mais ne sont jamais acceptés. On voit notamment que ton nœud n’accepte pas ce bloc vu qu’il re-génère plusieurs fois le même bloc (même n°).

Donc ton nœud ne forke pas à proprement parler, mais il patauge pour générer son bloc.

---

<div class="post-metadata">

### Author: ![cgeek](https://forum.duniter.org/user_avatar/forum.duniter.org/cgeek/32/279_2.png) [@cgeek](https://forum.duniter.org/u/cgeek)
#### Post date: [10 November 2017 08:05 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/11 "2017-11-10T08:05:01Z")

</div>

Bon, comme les tests ne sont pas encore très faciles à appréhender et que c’est une partie qui me tient à cœur, j’ai rédigé le test : [https://github.com/duniter/duniter/commit/d752c1cdddc8c2e8d851f3c73aac773d2c321887](https://github.com/duniter/duniter/commit/d752c1cdddc8c2e8d851f3c73aac773d2c321887)

Le test ne passait pas avant la correction d’Eloïs, puis passe une fois celle-ci appliquée. Pour vous en convaincre, changez la ligne 65 du fichier blockGenerator.ts en ceci :

```auto
const wereExcludeds:string[] = []

```

Ce qui revient à annuler la correction. Alors, le test ne passera plus : on verra Duniter tenter d’inclure une clé révoquée dans le champ `joiners` du bloc. Puis si l’on remet la correction, le test passe de nouveau, le champ `joiners` est vide comme attendu.

En fait, c’est le cas où une clé révoquée demande à redevenir membre : HydenStark1 a révoqué son compte et tente de revenir, certainement en cliquant sur « Renouveler mon adhésion » dans un client. Malheureusement c’est trop tard pour ce compte-là, il lui faut en recréer un nouveau.

---

<div class="post-metadata">

### Author: ![Inso](https://forum.duniter.org/user_avatar/forum.duniter.org/inso/32/1229_2.png) [@Inso](https://forum.duniter.org/u/Inso)
#### Post date: [10 November 2017 08:49 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/12 "2017-11-10T08:49:01Z")

</div>

Ça cache probablement un bug dans un client d’ailleurs 🙂 si quelqu’un sait qui est HydenStark et quel client il utilise…

---

<div class="post-metadata">

### Author: ![gerard94](https://forum.duniter.org/user_avatar/forum.duniter.org/gerard94/32/58_2.png) [@gerard94](https://forum.duniter.org/u/gerard94)
#### Post date: [10 November 2017 11:38 UTC](https://forum.duniter.org/t/comportement-bizarre-de-duniter-1-6-11/3592/13 "2017-11-10T11:38:09Z")

</div>

Au moins, c’est réconfortant.
