Skip to content

Commit 4d3dc34

Browse files
[management] remove duplicated removal of groups on peer delete (#4421)
1 parent 6fc50a4 commit 4d3dc34

File tree

2 files changed

+0
-6
lines changed

2 files changed

+0
-6
lines changed

management/server/grpcserver.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -999,8 +999,6 @@ func (s *GRPCServer) Logout(ctx context.Context, req *proto.EncryptedMessage) (*
999999
return nil, mapError(ctx, err)
10001000
}
10011001

1002-
s.accountManager.BufferUpdateAccountPeers(ctx, peer.AccountID)
1003-
10041002
log.WithContext(ctx).Debugf("peer %s logged out successfully after %s", peerKey.String(), time.Since(start))
10051003

10061004
return &proto.Empty{}, nil

management/server/peer.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -368,10 +368,6 @@ func (am *DefaultAccountManager) DeletePeer(ctx context.Context, accountID, peer
368368
return err
369369
}
370370

371-
if err = transaction.RemovePeerFromAllGroups(ctx, peer.ID); err != nil {
372-
return fmt.Errorf("failed to remove peer from groups: %w", err)
373-
}
374-
375371
eventsToStore, err = deletePeers(ctx, am, transaction, accountID, userID, []*nbpeer.Peer{peer})
376372
if err != nil {
377373
return fmt.Errorf("failed to delete peer: %w", err)

0 commit comments

Comments
 (0)