Ability to control headers

[harryqt]

Right now we are forced to use header: 'Cache-Control' => "max-age=$ttl, public", sometimes that's not what we want.

In my case, I wanted to use different header: 'Cache-Control' => 'max-age=0, s-maxage=600'.

Proposal

We can add a third parameter headers in cache() with same default type and value as $tags. If headers is not null then we use that else use the default.

class CloudflareRouterMixin
{
    public function cache(): Closure
    {
        return function (
            string|array|Closure $tags = null,
            int $ttl = null,
            string|array|Closure $headers = null,
        ) {
            //..
        };
    }
}

Route::cache(headers: [
    'Cache-Control' => 'max-age=0, s-maxage=600',
    //..
])->group(function () {
    //...
});

[harryqt]

@emredipi Can we make the header programable.?

[emredipi]

Currently, since cache parameters (tags, ttl) are passing into middleware directly as middleware parameters we can pass only string types. So we may not send a custom header but maybe we can add max-age parameter if that's a need.

[harryqt]

For future readers: you can use middleware to control the headers.

app/Http/Middleware/SetCacheControlHeader.php

<?php

declare(strict_types=1);

namespace App\Http\Middleware;

use Closure;
use Illuminate\Http\Request;
use Symfony\Component\HttpFoundation\Response;
use Yediyuz\CloudflareCache\CloudflarePagesMiddleware;

class SetCacheControlHeader
{
    /**
     * Handle an incoming request.
     *
     * @param  \Closure(\Illuminate\Http\Request): (\Symfony\Component\HttpFoundation\Response)  $next
     */
    public function handle(Request $request, Closure $next): Response
    {
        /** @var Response $response */
        $response = $next($request);

        if ((new CloudflarePagesMiddleware)->shouldCacheResponse($request, $response)) {
            $response->headers->set('Cache-Control', 'max-age=0, s-maxage=2592000');
        }

        return $response;
    }
}

routes/web.php

Route::middleware([SetCacheControlHeader::class])->group(function () {
    //...
});