Skip to content
This repository has been archived by the owner on Jun 22, 2023. It is now read-only.

Commit

Permalink
Merge pull request #212 from lidofinance/feat/audit_ack_broadcast_reinit
Browse files Browse the repository at this point in the history
LDC-07 Messages May Be Sent to a Single Node
  • Loading branch information
sergeyWh1te authored Mar 1, 2023
2 parents 7baa819 + a7e6856 commit 6e233ce
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions client/services/node/node_service.go
Original file line number Diff line number Diff line change
Expand Up @@ -548,6 +548,16 @@ func (s *BaseNodeService) reinitDKG(message storage.Message) error {
if fsm.Event(msg.Event) == sif.EventSigningStart {
break
}

// LDC-07 Messages May Be Sent to a Single Node
//
// If we remove the broadcast and send only individual messages,
// then this behavior will break backward compatibility
// with the log of the previous version of the application. (v0.1.4)
//
// The described case will not lead to a catastrophic scenario,
// maximum inconvenience, and restart of the procedure,
// which is not very scary compared to the loss of compatibility.
if msg.RecipientAddr == "" || msg.RecipientAddr == s.GetUsername() {
operation, err := s.processMessage(msg)
if err != nil {
Expand Down

0 comments on commit 6e233ce

Please sign in to comment.