Doesn't add etag when returning custom response about fastapi-etag

[lazka]

You could add the headers from response dependency to your new response like this, should work:

Thanks! I'll give that a try.

This worked nicely, thanks again

[lazka]

I've solved this with a middleware for now:

def get_etag():
    return "foo"

@app.middleware("http")
async def add_etags_everywhere(request: Request, call_next: Callable) -> Response:
    etag = request.headers.get("if-none-match")
    current_etag = get_etag()
    if etag == current_etag:
        return Response("", 304, headers={"etag": current_etag})
    response: Response = await call_next(request)
    if "etag" not in response.headers:
        response.headers["etag"] = current_etag
    return response

[steinitzu]

The problem is that returning a response object from the route completely overrides any changes to the response made in the dependency chain.

You could add the headers from response dependency to your new response like this, should work:

app.get('/foobar', dependencies=[Depends(Etag(get_etag))])
async def repos(request: Request, response: Response) -> Any:
    return JSONResponse([], headers=response.headers)

But the middleware approach is probably better if you're adding etags the same way on every route

[lazka]

You could add the headers from response dependency to your new response like this, should work:

Thanks! I'll give that a try.

Feel free to close this (and maybe add this to the doc?, btw the examples in the docs don't work due to missing Depends usage and import)

But the middleware approach is probably better if you're adding etags the same way on every route

That sadly didn't work out because I hit tiangolo/fastapi#1472