Fix rethrow policy

UnknownMiniProtocol errors, even on the outbound side, should not trigger a node shutdown.
IntersectMBO · Feb 3, 2025 · 653d62e · 653d62e
1 parent e91d5c4
commit 653d62e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/ouroboros-network/CHANGELOG.md b/ouroboros-network/CHANGELOG.md
@@ -25,6 +25,7 @@
 * Use `LocalRootConfig` instead of a tuple.
 * Extended `LocalRootConfig` with `diffusionMode :: DiffusionMode` field.
 * Added `diConnStateSupply` record field to `Ouroboros.Network.Diffusion.P2P.Interfaces`.
+* UnknownMiniProtocol error should not crash the node
 
 ### Non-Breaking changes
 

diff --git a/ouroboros-network/src/Ouroboros/Network/Diffusion/P2P.hs b/ouroboros-network/src/Ouroboros/Network/Diffusion/P2P.hs
@@ -745,11 +745,8 @@ runM Interfaces
           Nothing -> mempty)
       <>
       RethrowPolicy (\ctx err -> case  (ctx, fromException err) of
-                        -- mux unknown mini-protocol errors on the outbound
-                        -- side are fatal, since this is misconfiguration of the
-                        -- ouroboros-network stack.
                         (OutboundError, Just Mx.UnknownMiniProtocol {})
-                          -> ShutdownNode
+                          -> ShutdownPeer
                         _ -> mempty)