(Non-TLS) server should use `System.TimeManager.withManager` instead of `initManager`

well-typed / grapesy

Native Haskell gRPC client and server based on `http2`

Other

37 stars 4 forks source link

(Non-TLS) server should use `System.TimeManager.withManager` instead of `initManager` #190

Closed edsko closed 1 month ago

edsko commented 1 month ago

We currently use initManager, which means that we might leak some resources if the server is shut down. However, when we use withManager instead, the test suite fails:

    Test.Sanity.Interop
      cancellation
        client: FAIL
          Exception: KilledByHttp2ThreadManager (Just (BadThingHappen Network.Socket.sendBuf: resource vanished (Broken pipe)))

Currently unclear why.

edsko commented 1 month ago

Possibly related: #156.