Skip to content

feat(http): support HTTP/1.1 pipelining behind feature flag#3725

Draft
Syedowais312 wants to merge 2 commits intokeploy:mainfrom
Syedowais312:http-pipelining-recorder
Draft

feat(http): support HTTP/1.1 pipelining behind feature flag#3725
Syedowais312 wants to merge 2 commits intokeploy:mainfrom
Syedowais312:http-pipelining-recorder

Conversation

@Syedowais312
Copy link

Summary

Adds HTTP/1.1 pipelining support to the HTTP recorder by decoupling request and response I/O per connection and pairing responses using FIFO ordering.

Details

  • Uses a per-connection in-flight request queue
  • Responses are matched in FIFO order (as guaranteed by HTTP/1.1)
  • No request/response mutation
  • Guarded behind KEPLOY_HTTP_PIPELINING_ENABLED
  • Existing behavior remains unchanged by default

issue: #3724

func isHTTPPipeliningEnabled() bool {
return os.Getenv("KEPLOY_HTTP_PIPELINING_ENABLED") == "true"
}

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Legacy synchronous HTTP recording path.
Kept for compatibility until the pipelined implementation is validated.

Signed-off-by: syedowais312 <syedowais312sf@gmail.com>
@Syedowais312 Syedowais312 force-pushed the http-pipelining-recorder branch from d95685d to 6ddaa2f Compare February 5, 2026 15:30
@Syedowais312
Copy link
Author

Hey @gouravkrosx @Sarthak160 @slayerjain,
whenever you have time, could you please take a look at this PR?
Would love to get your feedback.

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