From c2a39d697faf9ebbdbec9cb99dbeed37e4db1316 Mon Sep 17 00:00:00 2001
From: Hosnain Rafi <rhosnain@gmail.com>
Date: Sun, 16 Aug 2026 19:59:41 +0000
Subject: [PATCH] fix: trim whitespace in redirect from and to

Redirects silently failed to match when a leading/trailing space was
present in the address (e.g. `to = " https://example.com"`), which is a
common typo that is hard to spot. Trimming the values in the redirect
normalizer resolves the issue while preserving the parsed rule shape.

Fixes https://github.com/netlify/cli/issues/4707
---
 src/utils/redirects.ts             | 11 ++++++++++-
 tests/unit/utils/redirects.test.ts | 26 ++++++++++++++++++++++++++
 2 files changed, 36 insertions(+), 1 deletion(-)

diff --git a/src/utils/redirects.ts b/src/utils/redirects.ts
index ffa925e1a8d..9f7a70d740a 100644
--- a/src/utils/redirects.ts
+++ b/src/utils/redirects.ts
@@ -36,6 +36,11 @@ const getErrorMessage = function ({ message }) {
 //  - `from` is called `origin`
 //  - `query` is called `params`
 //  - `conditions.role|country|language` are capitalized
+// Leading and trailing whitespace in `from` and `to` is trimmed so that typos
+// such as `to = " https://example.com"` do not silently break redirects
+// (see https://github.com/netlify/cli/issues/4707).
+const trimValue = (value) => (typeof value === 'string' ? value.trim() : value)
+
 const normalizeRedirect = function ({
   // @ts-expect-error TS(7031) FIXME: Binding element 'country' implicitly has an 'any' ... Remove this comment to see the full error message
   conditions: { country, language, role, ...conditions },
@@ -45,11 +50,15 @@ const normalizeRedirect = function ({
   query,
   // @ts-expect-error TS(7031) FIXME: Binding element 'signed' implicitly has an 'any' t... Remove this comment to see the full error message
   signed,
+  // @ts-expect-error TS(7031) FIXME: Binding element 'to' implicitly has an 'any type...
+  to,
   ...redirect
 }) {
   return {
     ...redirect,
-    origin: from,
+    origin: trimValue(from),
+    path: trimValue(from),
+    to: trimValue(to),
     params: query,
     conditions: {
       ...conditions,
diff --git a/tests/unit/utils/redirects.test.ts b/tests/unit/utils/redirects.test.ts
index 31492fd5da0..4484af07778 100644
--- a/tests/unit/utils/redirects.test.ts
+++ b/tests/unit/utils/redirects.test.ts
@@ -230,3 +230,29 @@ test('should parse redirect rules from _redirects file and netlify.toml', async
     expect(redirects).toEqual(expected)
   })
 })
+
+test('should trim leading and trailing whitespace from redirect `from` and `to`', async (t) => {
+  await withSiteBuilder(t, async (builder) => {
+    await builder
+      .withNetlifyToml({
+        config: {
+          redirects: [
+            {
+              from: ' /leading-space ',
+              status: 200,
+              to: ' https://www.netlify.com ',
+            },
+          ],
+        },
+      })
+      .build()
+
+    // @ts-expect-error TS(2345) FIXME: Argument of type '{ configPath: string; }' is not ... Remove this comment to see the full error message
+    const redirects = await parseRedirects({ configPath: `${builder.directory}/netlify.toml` })
+    expect(redirects[0]).toMatchObject({
+      origin: '/leading-space',
+      path: '/leading-space',
+      to: 'https://www.netlify.com',
+    })
+  })
+})