Static Caching - Content type not saved, always delivered as text/html

[andjsch]

Bug Description

With static caching enabled, as soon as a page is loaded the second time, the set content type is ignored. Instead, its content type is always text/html. See a sitemap as an example

How to Reproduce

Create a route that produces xml and enable static caching. See the above.

Route::statamic('/{loc}/sitemap.xml', 'sitemap', ['layout' => null, 'content_type' => 'application/xml'])->name('sitemap');

Extra Detail

first load:

second load onwards:

Environment

Laravel Valet

Statamic version: 3.0.0

PHP version: 7.4

Install method (choose one):
Used statamic install

[jasonvarga]

Full or half caching? It can't work for full.

[andjsch]

Full caching. Why can't it work for full caching? Isn't it possible to somehow save the content type in there?

[jasonvarga]

Full measure just saves html files and uses the server rewrite rule to look for those specifically named html files.

[andjsch]

I see. But then it should at least not cache the ones that are of a different content type

[jasonvarga]

That's fair.

You can exclude it from caching manually as a workaround though.
https://statamic.dev/static-caching#excluding-pages

[andjsch]

I actually did. It is a multi site which has the urls like this: domain.tld/{locale}/sitemap.xml

I excluded the sitemaps the following way: /*/sitemap.xml - this ain't working though

[jasonvarga]

Can't use wildcards there. Add them all individually.

[andjsch]

Ah, I see. Any plans to add wildcard support? Should I open an issue for that?

[jasonvarga]

You can open an issue if you want. Although I would say the more important thing is to be able to have better multisite support for static caching. I'm sure issues like that already exist.

[sjclark]

Full measure just saves html files and uses the server rewrite rule to look for those specifically named html files.

Just stumbled across this myself (using half caching), eventually figured out excludes before finding this thread. Exact same scenario (sitemap.xml).

Very possible I'm missing something..... but if content that is statically cached is always delivered as text/html then would it not make sense to restrict the function only to files/responses that are text/html?

Presumably there must be some sort of checking as things like by json files/routes don't seem to be converted when static caching?

[aerni]

I just ran into this issue as well. In my case, it's an atom feed that shouldn't be cached statically. It would be nice if non-html content types were excluded by default.

This is my route for reference:

Site::all()->each(function ($site) {
    Route::statamic("{$site->url()}/feed", 'feed/default', [
        'layout' => 'feed/layout',
        'content_type' => 'atom',
        'title' => 'News Feed',
    ]);
});

[aerni]

I tested this with both half and full measure static caching. It's the same issue with both caching methods.