remove Vite static asset middleware (sveltejs#4535)

* remove Vite static asset middleware * Update packages/kit/test/apps/basics/test/test.js Co-authored-by: Mihael Mutić <35368241+PH4NTOMiki@users.noreply.github.com> * alternative approach * update descriptions Co-authored-by: Mihael Mutić <35368241+PH4NTOMiki@users.noreply.github.com>
rmunn · Apr 6, 2022 · 32a7ccf · 32a7ccf
1 parent 1330b85
commit 32a7ccf
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/.changeset/shaggy-geckos-chew.md b/.changeset/shaggy-geckos-chew.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+Remove `static` directory from Vite's control
diff --git a/packages/kit/src/core/dev/index.js b/packages/kit/src/core/dev/index.js
@@ -28,7 +28,6 @@ export async function dev({ cwd, port, host, https, config }) {
 				fs: {
 					allow: [
 						...new Set([
-							config.kit.files.assets,
 							config.kit.files.lib,
 							config.kit.files.routes,
 							config.kit.outDir,

diff --git a/packages/kit/test/apps/basics/test/test.js b/packages/kit/test/apps/basics/test/test.js
@@ -2232,6 +2232,11 @@ test.describe.parallel('Static files', () => {
 		response = await request.get('/favicon.ico');
 		expect(response.status()).toBe(200);
 	});
+
+	test('does not use Vite to serve contents of static directory', async ({ request }) => {
+		const response = await request.get('/static/static.json');
+		expect(response.status()).toBe(process.env.DEV ? 403 : 404);
+	});
 });
 
 test.describe.parallel('Matchers', () => {