Skip to content

Conversation

@madroach
Copy link
Contributor

This is a proposal to make the semantics of connection cache and Net.ctx in the Cohttp_lwt.Client module clearer.
This change is essentially summarized in cohttp-lwt/src/client.mli:

 module Make (Connection : S.Connection) :
-  S.Client with type ctx = Connection.Net.ctx
+  S.Client with type ctx = S.call

This is just a proof-of-concept on how to fix #1074

Since a connection cache already includes Net.ctx, this makes the semantics of the Client module clearer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant