Skip to content

Commit

Permalink
federator: Do no reuse connections when talking to remotes
Browse files Browse the repository at this point in the history
This comes with performance penalty but its required to get around this bug in
the http2 library: kazu-yamamoto/http2#102
  • Loading branch information
akshaymankar committed Dec 20, 2023
1 parent 5357c57 commit b7b258c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion services/federator/src/Federator/Remote.hs
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ interpretRemote = interpret $ \case
resp <- mapError (RemoteError target pathT) . (fromEither @FederatorClientHTTP2Error =<<) . embed $
Codensity $ \k ->
E.catches
(H2Manager.withHTTP2Request mgr (True, hostname, fromIntegral port) req' (consumeStreamingResponseWith $ k . Right))
(H2Manager.withHTTP2RequestOnSingleUseConn mgr (True, hostname, fromIntegral port) req' (consumeStreamingResponseWith $ k . Right))
[ E.Handler $ k . Left,
E.Handler $ k . Left . FederatorClientTLSException,
E.Handler $ k . Left . FederatorClientHTTP2Exception,
Expand Down

0 comments on commit b7b258c

Please sign in to comment.