From d1e0b7f4fb45339197f567602f91bf4a943dcaf0 Mon Sep 17 00:00:00 2001
From: Pascal Fischer <32096965+pascal-fischer@users.noreply.github.com>
Date: Tue, 5 Aug 2025 01:11:44 +0200
Subject: [PATCH] [management] get peer groups without lock (#4280)

---
 management/server/ephemeral_test.go | 2 +-
 management/server/peer.go           | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/management/server/ephemeral_test.go b/management/server/ephemeral_test.go
index f71d48c58..d07b9a422 100644
--- a/management/server/ephemeral_test.go
+++ b/management/server/ephemeral_test.go
@@ -43,10 +43,10 @@ func (a *MockAccountManager) DeletePeer(_ context.Context, accountID, peerID, us
 	a.mu.Lock()
 	defer a.mu.Unlock()
 	a.deletePeerCalls++
+	delete(a.store.account.Peers, peerID)
 	if a.wg != nil {
 		a.wg.Done()
 	}
-	delete(a.store.account.Peers, peerID)
 	return nil
 }
 
diff --git a/management/server/peer.go b/management/server/peer.go
index f954369d8..ddf54bd59 100644
--- a/management/server/peer.go
+++ b/management/server/peer.go
@@ -1525,7 +1525,7 @@ func (am *DefaultAccountManager) getInactivePeers(ctx context.Context, accountID
 
 // GetPeerGroups returns groups that the peer is part of.
 func (am *DefaultAccountManager) GetPeerGroups(ctx context.Context, accountID, peerID string) ([]*types.Group, error) {
-	return am.Store.GetPeerGroups(ctx, store.LockingStrengthShare, accountID, peerID)
+	return am.Store.GetPeerGroups(ctx, store.LockingStrengthNone, accountID, peerID)
 }
 
 // getPeerGroupIDs returns the IDs of the groups that the peer is part of.