Skip to content

Commit 1668e11

Browse files
Merge remote-tracking branch 'pokt/main' into refactor/consolidate-p2p-modules
* pokt/main: [Consensus] Log warnings if server module is not enabled (#679)
2 parents 09d173a + a7fd2a6 commit 1668e11

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

consensus/doc/CHANGELOG.md

+5-1
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
77

88
## [Unreleased]
99

10-
## [0.0.0.47] - 2023-04-17
10+
## [0.0.0.48] - 2023-04-17
1111

1212
- Debug logging improvements
1313

14+
## [0.0.0.47] - 2023-04-17
15+
16+
- Log warnings in `handleStateSyncMessage()`
17+
1418
## [0.0.0.46] - 2023-04-13
1519

1620
- Utilise the `TxResult` protobuf from `shared/core/types`

consensus/state_sync_handler.go

+4-2
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,8 @@ func (m *consensusModule) handleStateSyncMessage(stateSyncMessage *typesCons.Sta
3838
case *typesCons.StateSyncMessage_MetadataReq:
3939
m.logger.Info().Str("proto_type", "MetadataRequest").Msg("Handling StateSyncMessage MetadataReq")
4040
if !m.serverModeEnabled {
41-
return fmt.Errorf("server module is not enabled")
41+
m.logger.Warn().Msg("Node's server module is not enabled")
42+
return nil
4243
}
4344
return m.stateSync.HandleStateSyncMetadataRequest(stateSyncMessage.GetMetadataReq())
4445
case *typesCons.StateSyncMessage_MetadataRes:
@@ -48,7 +49,8 @@ func (m *consensusModule) handleStateSyncMessage(stateSyncMessage *typesCons.Sta
4849
case *typesCons.StateSyncMessage_GetBlockReq:
4950
m.logger.Info().Str("proto_type", "GetBlockRequest").Msg("Handling StateSyncMessage GetBlockRequest")
5051
if !m.serverModeEnabled {
51-
return fmt.Errorf("server module is not enabled")
52+
m.logger.Warn().Msg("Node's server module is not enabled")
53+
return nil
5254
}
5355
return m.stateSync.HandleGetBlockRequest(stateSyncMessage.GetGetBlockReq())
5456
case *typesCons.StateSyncMessage_GetBlockRes:

0 commit comments

Comments
 (0)