[PR #255] [MERGED] fix: config redirects not applying to rsc requests #420

New issue

Closed

opened 2026-05-06 12:39:43 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 12:39:43 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cloudflare/vinext/pull/255
Author: @james-elicx
Created: 3/5/2026
Status: ✅ Merged
Merged: 3/5/2026
Merged by: @james-elicx

Base: main ← Head: james/config-redirect-rsc

📝 Commits (5)

f3a0cc3 fix: config redirects not applying to rsc requests
9cd487c fix poorly written test
6924886 add better fix for soft nav url diffs
1e9c7c7 escape the .
9a7cb49 fix: strip .rsc from beforeFiles rewrites; fix redirect recursion and double pushState

📊 Changes

5 files changed (+81 additions, -4 deletions)

View changed files

📝 packages/vinext/src/server/app-dev-server.ts (+28 -3)
📝 tests/app-router.test.ts (+23 -1)
📝 tests/e2e/app-router/config-redirect.spec.ts (+21 -0)
📝 tests/fixtures/app-basic/app/page.tsx (+3 -0)
📝 tests/fixtures/app-basic/next.config.ts (+6 -0)

📄 Description

soft-nav and hard-nav to rsc paths don't apply redirects defined in the config due to the .rsc extension not matching the configured path.

further fix to update the browser url to match the redirected url on soft navs.

_{🔄 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/255 **Author:** [@james-elicx](https://github.com/james-elicx) **Created:** 3/5/2026 **Status:** ✅ Merged **Merged:** 3/5/2026 **Merged by:** [@james-elicx](https://github.com/james-elicx) **Base:** `main` ← **Head:** `james/config-redirect-rsc` --- ### 📝 Commits (5) - [`f3a0cc3`](https://github.com/cloudflare/vinext/commit/f3a0cc3d4101e6e04dcd4413d4dd54e032be4f99) fix: config redirects not applying to rsc requests - [`9cd487c`](https://github.com/cloudflare/vinext/commit/9cd487c7ebb1aa247df7a7d6c8e367a32a84002d) fix poorly written test - [`6924886`](https://github.com/cloudflare/vinext/commit/69248863e162bd2cab6037224d21a95ceb0ce008) add better fix for soft nav url diffs - [`1e9c7c7`](https://github.com/cloudflare/vinext/commit/1e9c7c7f28bdb3e237a56a657f49ab76141043a9) escape the . - [`9a7cb49`](https://github.com/cloudflare/vinext/commit/9a7cb49c8f312bd59abbeaba3c76ed8759efbee5) fix: strip .rsc from beforeFiles rewrites; fix redirect recursion and double pushState ### 📊 Changes **5 files changed** (+81 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `packages/vinext/src/server/app-dev-server.ts` (+28 -3) 📝 `tests/app-router.test.ts` (+23 -1) 📝 `tests/e2e/app-router/config-redirect.spec.ts` (+21 -0) 📝 `tests/fixtures/app-basic/app/page.tsx` (+3 -0) 📝 `tests/fixtures/app-basic/next.config.ts` (+6 -0) </details> ### 📄 Description soft-nav and hard-nav to rsc paths don't apply redirects defined in the config due to the `.rsc` extension not matching the configured path. further fix to update the browser url to match the redirected url on soft navs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>