Updates SSR routing to always give priority to public assets (#4000)

* matchRoute should ignore requests for public assets

* chore: add changeset

1 files changed, 11 insertions, 0 deletions

diff --git a/packages/astro/test/ssr-dynamic.test.js b/packages/astro/test/ssr-dynamic.test.js
index 8c200f1c5..f2795794f 100644
--- a/packages/astro/test/ssr-dynamic.test.js
+++ b/packages/astro/test/ssr-dynamic.test.js
@@ -18,6 +18,12 @@ describe('Dynamic pages in SSR', () => {
 		await fixture.build();
 	});
 
+	async function matchRoute(path) {
+		const app = await fixture.loadTestAdapterApp();
+		const request = new Request('https://example.com' + path);
+		return app.match(request);
+	}
+
 	async function fetchHTML(path) {
 		const app = await fixture.loadTestAdapterApp();
 		const request = new Request('http://example.com' + path);
@@ -50,4 +56,9 @@ describe('Dynamic pages in SSR', () => {
 		const json = await fetchJSON('/api/products/33');
 		expect(json.id).to.equal('33');
 	});
+
+	it('Public assets take priority', async () => {
+		const favicon = await matchRoute('/favicon.ico');
+		expect(favicon).to.equal(undefined);
+	});
 });