Tee Ming
Tee Ming
I'd go one step further and prevent generating a fallback if there are no prerendered pages in the cloudflare adapter. The main issue was invalid requests to `/_app/**` incorrectly returned...
> Would dealing with #9802 remove the need for this? It would be nice not to need another option. I believe it would. The only reason for this option was...
The proposed solution actually implements a static 404.html by default too. This would be without running hooks
> It would implement it for the appDir, not the actual app. The fallback page would still be needed for the actual app to ensure that 404s in the app...
closed by https://github.com/sveltejs/kit/pull/11596 which now does not generate a fallback by default and provides the ability to do so through the adapter config.
> Have we thought about the effects this might have on [kit.svelte.dev/docs/routing#server-content-negotiation](https://kit.svelte.dev/docs/routing#server-content-negotiation) ? Content negotiation is still in play here, so the `accept: text/html` always connects the page. > Also,...
> > Also, if we decided (as in #9780) to start adding `Vary: Accept` to responses for requests with sibling `GET`/`POST` endpoints and pages, would this make that more difficult?...
reverted my previous merge with master and did it with rebase instead.
Oh whoops, I completely missed that. I'll look into it now.
I've updated the PR for `GET` to take precedence over `all` for `HEAD` requests.