From f9992c3ac8a9cbf62643d17844da1e2c9d2ccced Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Zolt=C3=A1n=20Papp?= <zoltan.pmail@gmail.com>
Date: Mon, 29 Sep 2025 21:36:04 +0200
Subject: [PATCH] Fix sonar issue

---
 client/internal/peer/signaler.go | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/client/internal/peer/signaler.go b/client/internal/peer/signaler.go
index 7624597fa..40f4bc561 100644
--- a/client/internal/peer/signaler.go
+++ b/client/internal/peer/signaler.go
@@ -86,15 +86,7 @@ func (s *Signaler) signalOfferAnswer(offerAnswer OfferAnswer, remoteKey string,
 		case errors.Is(err, signal.ErrPeerNotAvailable):
 			return ErrPeerNotAvailable
 		case errors.Is(err, signal.ErrUnimplementedMethod):
-			// print out the warning only once
-			if !s.deliveryCheckNotSupported.Load() {
-				log.Warnf("signal client does not support delivery check, falling back to Send method and resend")
-			}
-
-			s.deliveryCheckNotSupported.Store(true)
-			if err := s.signal.Send(msg); err != nil {
-				log.Warnf("failed to send signal msg to remote peer: %v", err)
-			}
+			s.handleUnimplementedMethod(msg)
 			return ErrSignalNotSupportDeliveryCheck
 		default:
 			return err
@@ -112,3 +104,15 @@ func (s *Signaler) SignalIdle(remoteKey string) error {
 		},
 	})
 }
+
+func (s *Signaler) handleUnimplementedMethod(msg *sProto.Message) {
+	// print out the warning only once
+	if !s.deliveryCheckNotSupported.Load() {
+		log.Warnf("signal client does not support delivery check, falling back to Send method and resend")
+	}
+
+	s.deliveryCheckNotSupported.Store(true)
+	if err := s.signal.Send(msg); err != nil {
+		log.Warnf("failed to send signal msg to remote peer: %v", err)
+	}
+}