Skip to content

fix(httpaf-effects): fix build error and runtime connection handling#38

Open
MrSagarBiswas wants to merge 1 commit into
ocaml-multicore:masterfrom
MrSagarBiswas:master
Open

fix(httpaf-effects): fix build error and runtime connection handling#38
MrSagarBiswas wants to merge 1 commit into
ocaml-multicore:masterfrom
MrSagarBiswas:master

Conversation

@MrSagarBiswas

@MrSagarBiswas MrSagarBiswas commented Mar 21, 2026

Copy link
Copy Markdown

Summary

This PR fixes the remaining httpaf-effects stability problems with minimal, focused changes.

  • Fixes the partial application build failure in the benchmark handler path.
  • Fixes connection I/O handling so requests are read and written correctly.
  • Uses a bounded read buffer to avoid out-of-memory crashes.

Result

  • Build succeeds.
  • Runtime no longer fails with prompt input shrunk, connection reset, or OOM for normal requests.
  • Endpoint responds correctly to curl with HTTP 200.

Issues

Fixes #35
Fixes #36
Fixes #37

Issue reference: #37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant