fix:Configuring server.baseURL will make the api route inaccessible #1871
Conversation
|
✅ Deploy Preview for solid-start-landing-page ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Changeset missing |
|
|
||
| // api | ||
| const match = matchAPIRoute(new URL(event.request.url).pathname, event.request.method); | ||
| const match = matchAPIRoute(event.nativeEvent.path, event.request.method); |
There was a problem hiding this comment.
The event.nativeEvent.path includes the query string 1 and the matchAPIRoute won't work in this case.
Maybe:
const [pathname] = event.nativeEvent.path.split("?", 2);
const match = matchAPIRoute(pathname, event.request.method);
There was a problem hiding this comment.
so your saying this won't work? @thislight
More templates
commit: |
PR Checklist
Please check if your PR fulfills the following requirements:
server.baseURLwill make theapiroute inaccessible #1857What is the current behavior?
api route return 404
What is the new behavior?
api route return 200
Other information
none