Skip to content

feat(middleware): define HTTP response pre-return interface - #3073

Draft
pimlock wants to merge 2 commits into
2691-middleware-header-authority/pimlockfrom
2691-http-response-interface/pimlock
Draft

feat(middleware): define HTTP response pre-return interface#3073
pimlock wants to merge 2 commits into
2691-middleware-header-authority/pimlockfrom
2691-http-response-interface/pimlock

Conversation

@pimlock

@pimlock pimlock commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Define the public HTTP_RESPONSE/PRE_RETURN contract without enabling response processing in the proxy. This lets reviewers evaluate the protobuf lifecycle and transport shape independently from the runtime implementation.

Related Issue

Part of #2691

Changes

  • Add the bidirectional HttpResponsePreReturn.Evaluate service.
  • Define preflight, body unit, body acknowledgement, body end, trailer, and typed session-end messages.
  • Accept the HTTP_RESPONSE/PRE_RETURN manifest binding and expose response-chain description.
  • Add transport-neutral in-process and remote gRPC stream interfaces.
  • Add focused manifest validation coverage.

Testing

Checklist

Stack

  1. feat(middleware): broaden HTTP header mutation authority #3072 — shared header mutation authority
  2. feat(middleware): define HTTP response pre-return interface #3073 — response protobuf and transport interface
  3. feat(network): inspect HTTP responses before return #3074 — response runtime, relay integration, and existing example
  4. docs(middleware): document response pre-return workflow #3075 — documentation and middleware guide reorganization

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Sep 1, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
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