Make Netlify adapter actually append redirects (#3079)

withastro · Apr 11, 2022 · 9f248b0 · 9f248b0
1 parent 771127e
commit 9f248b0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/.changeset/lemon-elephants-sell.md b/.changeset/lemon-elephants-sell.md
@@ -0,0 +1,5 @@
+---
+'@astrojs/netlify': patch
+---
+
+Make Netlify adapter actually append redirects
diff --git a/packages/integrations/netlify/src/index.ts b/packages/integrations/netlify/src/index.ts
@@ -53,11 +53,10 @@ ${pattern}    /.netlify/functions/${entryFile}    200`;
 					}
 				}
 
-				if (fs.existsSync(_redirects)) {
-					await fs.promises.appendFile(_redirectsURL, _redirects, 'utf-8');
-				} else {
-					await fs.promises.writeFile(_redirectsURL, _redirects, 'utf-8');
-				}
+				// Always use appendFile() because the redirects file could already exist,
+				// e.g. due to a `/public/_redirects` file that got copied to the output dir.
+				// If the file does not exist yet, appendFile() automatically creates it.
+				await fs.promises.appendFile(_redirectsURL, _redirects, 'utf-8');
 			},
 		},
 	};