From 635f170762c3fe0bd5ce4d338a97087c71fea576 Mon Sep 17 00:00:00 2001 From: William Petit Date: Fri, 5 Jan 2024 14:16:15 +0100 Subject: [PATCH] feat: only show warn in log when peer id is already used --- server/handler.go | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/server/handler.go b/server/handler.go index a9bc4e7..184991a 100644 --- a/server/handler.go +++ b/server/handler.go @@ -54,9 +54,8 @@ func AdvertiseHandler(store peering.Store, key *rsa.PublicKey, funcs ...OptionFu return } - peer, err := store.Get(serverClaims.PeerID) - if err == nil { - logger.Printf("[ERROR] %+v", errors.WithStack(ErrPeerIDAlreadyInUse)) + if _, err := store.Get(serverClaims.PeerID); err == nil { + logger.Printf("[WARN] %s", errors.WithStack(ErrPeerIDAlreadyInUse)) options.ErrorHandler(w, r, ErrPeerIDAlreadyInUse) return } @@ -69,7 +68,7 @@ func AdvertiseHandler(store peering.Store, key *rsa.PublicKey, funcs ...OptionFu attrs := filterAttributes(options.PeerAttributes, advertising.Attributes) - peer, err = store.Create(serverClaims.PeerID, attrs) + peer, err := store.Create(serverClaims.PeerID, attrs) if err != nil { logger.Printf("[ERROR] %+v", errors.WithStack(err)) options.ErrorHandler(w, r, err)