Merge pull request #6621 from AndriiDiachuk/observer-ignore-invalid-p…

…eers Changed log level not to spam messages
onflow · Nov 7, 2024 · f44c7fb · f44c7fb
2 parents 43ee3c5 + ed149a7
commit f44c7fb
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/cmd/observer/node_builder/observer_builder.go b/cmd/observer/node_builder/observer_builder.go
@@ -938,7 +938,7 @@ func (builder *ObserverServiceBuilder) InitIDProviders() {
 
 					if flowID, err := builder.IDTranslator.GetFlowID(pid); err != nil {
 						// TODO: this is an instance of "log error and continue with best effort" anti-pattern
-						builder.Logger.Err(err).Str("peer", p2plogging.PeerId(pid)).Msg("failed to translate to Flow ID")
+						builder.Logger.Debug().Str("peer", p2plogging.PeerId(pid)).Msg("failed to translate to Flow ID")
 					} else {
 						result = append(result, flowID)
 					}

diff --git a/cmd/utils.go b/cmd/utils.go
@@ -157,7 +157,7 @@ func CreatePublicIDTranslatorAndIdentifierProvider(
 
 				if flowID, err := idTranslator.GetFlowID(pid); err != nil {
 					// TODO: this is an instance of "log error and continue with best effort" anti-pattern
-					logger.Err(err).Str("peer", p2plogging.PeerId(pid)).Msg("failed to translate to Flow ID")
+					logger.Debug().Str("peer", p2plogging.PeerId(pid)).Msg("failed to translate to Flow ID")
 				} else {
 					result = append(result, flowID)
 				}

diff --git a/follower/follower_builder.go b/follower/follower_builder.go
@@ -461,7 +461,7 @@ func (builder *FollowerServiceBuilder) InitIDProviders() {
 
 					if flowID, err := builder.IDTranslator.GetFlowID(pid); err != nil {
 						// TODO: this is an instance of "log error and continue with best effort" anti-pattern
-						builder.Logger.Err(err).Str("peer", p2plogging.PeerId(pid)).Msg("failed to translate to Flow ID")
+						builder.Logger.Debug().Str("peer", p2plogging.PeerId(pid)).Msg("failed to translate to Flow ID")
 					} else {
 						result = append(result, flowID)
 					}