[PR #305] [MERGED] refactor: extract applyMiddlewareRequestHeaders into config-matchers #462

Closed
opened 2026-05-06 13:08:13 +02:00 by BreizhHardware · 0 comments

📋 Pull Request Information

Original PR: https://github.com/cloudflare/vinext/pull/305
Author: @james-elicx
Created: 3/6/2026
Status: Merged
Merged: 3/7/2026
Merged by: @james-elicx

Base: mainHead: refactor/extract-apply-middleware-request-headers


📝 Commits (2)

  • 16ec70a refactor: extract applyMiddlewareRequestHeaders into config-matchers
  • f10a781 fix(test): update deploy test to reflect applyMiddlewareRequestHeaders refactor

📊 Changes

4 files changed (+72 additions, -59 deletions)

View changed files

📝 packages/vinext/src/config/config-matchers.ts (+52 -0)
📝 packages/vinext/src/deploy.ts (+7 -32)
📝 packages/vinext/src/server/prod-server.ts (+8 -22)
📝 tests/deploy.test.ts (+5 -5)

📄 Description

Moves the duplicated x-middleware-request-* unpacking logic from prod-server.ts and deploy.ts into a shared applyMiddlewareRequestHeaders() in config-matchers.ts.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/cloudflare/vinext/pull/305 **Author:** [@james-elicx](https://github.com/james-elicx) **Created:** 3/6/2026 **Status:** ✅ Merged **Merged:** 3/7/2026 **Merged by:** [@james-elicx](https://github.com/james-elicx) **Base:** `main` ← **Head:** `refactor/extract-apply-middleware-request-headers` --- ### 📝 Commits (2) - [`16ec70a`](https://github.com/cloudflare/vinext/commit/16ec70a23d2b15b46ad972b21b6abbab35698e78) refactor: extract applyMiddlewareRequestHeaders into config-matchers - [`f10a781`](https://github.com/cloudflare/vinext/commit/f10a7813ec0499a11a591bcbed67ba2208a835cb) fix(test): update deploy test to reflect applyMiddlewareRequestHeaders refactor ### 📊 Changes **4 files changed** (+72 additions, -59 deletions) <details> <summary>View changed files</summary> 📝 `packages/vinext/src/config/config-matchers.ts` (+52 -0) 📝 `packages/vinext/src/deploy.ts` (+7 -32) 📝 `packages/vinext/src/server/prod-server.ts` (+8 -22) 📝 `tests/deploy.test.ts` (+5 -5) </details> ### 📄 Description Moves the duplicated x-middleware-request-* unpacking logic from `prod-server.ts` and `deploy.ts` into a shared `applyMiddlewareRequestHeaders()` in `config-matchers.ts`. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
BreizhHardware 2026-05-06 13:08:13 +02:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/vinext#462
No description provided.