Note: there is one edge-case the decorator isn't handling at the moment: if the decorated endpoint uses the names request or response but expects these to be a different type:

@app.get("/foo/{request}/{response}")
def foo(request: int, response: str) -> None:
    pass

This is perfectly valid, but the cache decorator will then fail with a ValueError: duplicate parameter name: 'request' or ValueError: duplicate parameter name: 'response' exception.

To make that work, you'd have to generate unique names for the request and response parameters. That's well outside the scope of this PR however.

Another issue is that the decorator assumes the request and response parameters always are named 'request' and 'response' when popping these from the copy_kwargs dict.

Thanks!