Skip to content

Commit

Permalink
Merge pull request #5063 from IntersectMBO/mwojtowicz/release/ourobor…
Browse files Browse the repository at this point in the history
…os-network-0.19

Fix rethrow policy
  • Loading branch information
crocodile-dentist authored Feb 3, 2025
2 parents 062b7b7 + 62e0887 commit 6065cff
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:

env:
# Modify this value to "invalidate" all cabal caches.
CABAL_CACHE_VERSION: "2024-05-22"
CABAL_CACHE_VERSION: "2025-02-03"
# Modify this value to "invalidate" the cabal store cache only.
CACHE_VERSION: "20220919"
# Modify this value to "invalidate" the dist-newstyle cache only.
Expand Down
6 changes: 6 additions & 0 deletions ouroboros-network/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,12 @@

## 0.19.0.2 -- 2025-02-03

### Breaking changes

### Non-breaking changes

* UnknownMiniProtocol error should not crash the node

## 0.19.0.1 -- 2025-01-15

### Breaking changes
Expand Down
5 changes: 1 addition & 4 deletions ouroboros-network/src/Ouroboros/Network/Diffusion/P2P.hs
Original file line number Diff line number Diff line change
Expand Up @@ -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)


Expand Down

0 comments on commit 6065cff

Please sign in to comment.