From c0a8c881600d11c63e90c660a8dba7131d7a6611 Mon Sep 17 00:00:00 2001 From: JJ Kasper Date: Thu, 3 Nov 2022 14:30:57 -0700 Subject: [PATCH] Fix undici warning showing unexpectedly (#42444) Fixes: https://github.com/vercel/next.js/issues/42441 ## Bug - [ ] Related issues linked using `fixes #number` - [ ] Integration tests added - [ ] Errors have a helpful link attached, see `contributing.md` ## Feature - [ ] Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR. - [ ] Related issues linked using `fixes #number` - [ ] Integration tests added - [ ] Documentation added - [ ] Telemetry added. In case of a feature if it's used or not. - [ ] Errors have a helpful link attached, see `contributing.md` ## Documentation / Examples - [ ] Make sure the linting passes by running `pnpm build && pnpm lint` - [ ] The "examples guidelines" are followed from [our contributing doc](https://github.com/vercel/next.js/blob/canary/contributing/examples/adding-examples.md) --- packages/next/server/config.ts | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/packages/next/server/config.ts b/packages/next/server/config.ts index af9f3eb80e2fc..400324e11c609 100644 --- a/packages/next/server/config.ts +++ b/packages/next/server/config.ts @@ -59,7 +59,11 @@ const experimentalWarning = execOnce( export function setHttpClientAndAgentOptions(config: NextConfig) { if (isAboveNodejs16) { - if (config.experimental?.enableUndici && isAboveNodejs18) { + if ( + config.experimental?.enableUndici && + !config.experimental?.appDir && + isAboveNodejs18 + ) { Log.warn( `\`enableUndici\` option is unnecessary in Node.js v${NODE_18_VERSION} or greater.` )