chore: migrate from middleware.ts to proxy.ts for Next.js 16 compatib… #8419
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/nodejs-website Please review the changes when you have a chance. Thank you! 🙏 |
There was a problem hiding this comment.
export const proxy = createMiddleware({
You should just be able to do this, no?
I have done as per documentation, but let me try if this similer approch works |
…ility Migrate from the deprecated middleware.ts file convention to the new proxy.ts convention as required by Next.js 16.0.10. The middleware file convention has been deprecated and renamed to 'proxy' to better clarify its purpose as a network boundary in front of the app, and to avoid confusion with Express.js middleware. Changes: - Rename apps/site/middleware.ts to apps/site/proxy.ts - Add named export function 'proxy' instead of default export - Add NextRequest type import for type safety - Maintain full compatibility with next-intl internationalization - Update comments to reflect proxy terminology This resolves the deprecation warning: 'The middleware file convention is deprecated. Please use proxy instead.' Refs: https://nextjs.org/docs/messages/middleware-to-proxy
62fb290 to
79378b7
Compare
There was a problem hiding this comment.
The reference to this file in CODEOWNERS needs updating, please
8000|
hi @avivkeller I amended the commit to incorporate this feedback. I've verified that both approaches work correctly, and all tests pass |
okay checking that |
hi @MattIPv4 I have done that please check if I have done it correctly or not |
|
The added comment is still not needed, you only removed half of it. |
yeah sorry about that check now f4507a8 |
Not concerns. This actually cannot be merged, otherwise it breaks Cloudflare builds. |
|
To be clear not against the changes, just, they cannot be merged meantime. |
Okay 👍, I will try to remind you once Open Next Supports it |
|
Thank you! You can keep the PR open although I'd recommend converting it back to a draft state. |
…ility
Migrate from the deprecated middleware.ts file convention to the new proxy.ts convention as required by Next.js 16.0.10.
The middleware file convention has been deprecated and renamed to 'proxy' to better clarify its purpose as a network boundary in front of the app, and to avoid confusion with Express.js middleware.
Changes:
This resolves the deprecation warning:
'The middleware file convention is deprecated. Please use proxy instead.'
Refs: https://nextjs.org/docs/messages/middleware-to-proxy
Description
Validation
Related Issues
Fixed #8418
Check List
pnpm formatto ensure the code follows the style guide.pnpm testto check if all tests are passing.pnpm buildto check if the website builds without errors.