Skip to content

feat(middleware): broaden HTTP header mutation authority - #3072

Draft
pimlock wants to merge 1 commit into
mainfrom
2691-middleware-header-authority/pimlock
Draft

feat(middleware): broaden HTTP header mutation authority#3072
pimlock wants to merge 1 commit into
mainfrom
2691-middleware-header-authority/pimlock

Conversation

@pimlock

@pimlock pimlock commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Remove the request-only header write namespace and centralize middleware header mutation checks. This is the base of the #2691 stack and keeps the authority change separate from response processing.

Related Issue

Part of #2691

Changes

  • Allow middleware to write permitted end-to-end request headers without an x-openshell-middleware- prefix.
  • Route writes and removals through one atomic validator and applicator.
  • Define request and response authority profiles with protected credential, routing, framing, connection, and security fields.
  • Add focused tests for shared limits, atomicity, direction-specific protection, and unrestricted end-to-end fields.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated, not applicable to this isolated validator change

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.

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