Skip to content
Snippets Groups Projects

[fix] #1200 Redefine unlock conditions

Merged Cédric Moreau requested to merge txunlock1.6 into 1.6

Voilà mon correctif pour le ticket #1200 (closed). C'est une partie sensible, puisqu'elle concerne le déverrouillage de sources.

Les tests précédents passent toujours, il y a même un test supplémentaire pour gérer le cas produit par Max sur le forum.

J'ai tenté de simplifier et centraliser au maximum le déverrouillage dans la fonction txunlock, ce qui devrait faciliter la lecture et la compréhension du code (je l'espère). En tout cas je profite de la livraison de la 1.6 pour passer ce correctif, car il me paraît vraiment très important.

Attention : si jamais un utilisateur s'amuse à passer sur Ğ1 une transaction avec des opérateurs OU, alors les nœuds en version 1.6.13 ou inférieur forkeront inévitablement. Pour l'instant il n'y a aucune transaction de ce type sur Ğ1, mais méfiance.

Qu'en pensez-vous ?

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Note : j'ai aussi testé la rétro-compatibilité avec la Ğ1 et la Ğ1-Test en faisant une synchro --cautious complète, et celle-ci est bien assurée. D'ailleurs ça faisait un moment que je ne l'avais pas faite, ce qui rassure quant au fait qu'il n'y a aucune anomalie de détectée sur ces blockchains.

  • Created by: coveralls

    Coverage Status

    Coverage decreased (-0.03%) to 79.255% when pulling c186ea8c on txunlock1.6 into 9fbe45d2 on 1.6.

  • So let's merge?

Please register or sign in to reply
Loading