michael/astro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fixes dynamic API routes in SSR (#3006)

Browse source 
* Fixes dynamic API routes in SSR

* Adds a changeset
This commit is contained in:
Matthew Phillips 2022-04-06 13:01:22 -04:00 committed by GitHub
parent 3bf5d84016
commit 68e1e2dd31
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 26 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
.changeset/three-donkeys-train.md Normal file
View file

@ -0,0 +1,5 @@
---
'astro': patch
---
Fixes dynamic API routes in SSR

3
packages/astro/src/core/app/index.ts
View file

@ -119,7 +119,7 @@ export class App {
async #callEndpoint(
request: Request,
_routeData: RouteData,
routeData: RouteData,
mod: ComponentInstance
): Promise<Response> {
const url = new URL(request.url);
@ -129,6 +129,7 @@ export class App {
origin: url.origin,
pathname: url.pathname,
request,
route: routeData,
routeCache: this.#routeCache,
ssr: true,
});

6
packages/astro/test/fixtures/ssr-dynamic/src/pages/api/products/[id].js vendored Normal file
View file

@ -0,0 +1,6 @@
export function get(params) {
return {
body: JSON.stringify(params)
};
}

13
packages/astro/test/ssr-dynamic.test.js
View file

@ -27,6 +27,14 @@ describe('Dynamic pages in SSR', () => {
return html;
}
async function fetchJSON(path) {
const app = await fixture.loadTestAdapterApp();
const request = new Request('http://example.com' + path);
const response = await app.render(request);
const json = await response.json();
return json;
}
it('Do not have to implement getStaticPaths', async () => {
const html = await fetchHTML('/123');
const $ = cheerioLoad(html);
@ -38,4 +46,9 @@ describe('Dynamic pages in SSR', () => {
const $ = cheerioLoad(html);
expect($('link').length).to.equal(1);
});
it('Dynamic API routes work', async () => {
const json = await fetchJSON('/api/products/33');
expect(json.id).to.equal('33');
});
});