No option to customise Cache-Control response header

[ermiyaeskandary]

I would like to set Cache-Control: no-cache to be returned for resources that should be revalidated with the server before every use (in addition to other parameters I set such as expiry.

However, the code currently does not allow any customisation and only sets max-age.

Note L201 in fastapi_cache>decorator.py:

if response:
    response.headers.update(
        {
            "Cache-Control": f"max-age={expire}",
            "ETag": f"W/{hash(to_cache)}",
            cache_status_header: "MISS",
        }
    )

"Cache-Control": f"max-age={expire}",

This prevents me from using the library in production environments as it can generate really state content...
If I submit a PR to allow customising this, will it kindly be reviewed and merged promptly considering the outstanding PR count?

Thank you!
EE

[ermiyaeskandary]

Similar to PR #71 in terms of the header that is affected

[ermiyaeskandary]

I've fixed this for now in my code with

api.py

app.add_middleware(CacheControlHeaderMiddleware)
...

CacheControlHeaderMiddleware.py

class CacheControlHeaderMiddleware(BaseHTTPMiddleware):
    async def dispatch(self, request, call_next):
        # https://github.com/long2ice/fastapi-cache/issues/498
        if 'Cache-Control' in response.headers:
            response.headers['Cache-Control'] = f"no-cache, {response.headers['Cache-Control']}"

        return response

[vicchi]

will it kindly be reviewed and merged promptly considering the outstanding PR count?

@ermiyaeskandary ... yes, I'm trying, not entirely successfully, to work my way through all of the PRs and triage them either for the next point release or for the v1 release; subject to spare time (hah!)