CacheMixin doesn't pass kwargs to ClientSession when Sentry is used about aiohttp-client-cache HOT 5 CLOSED

Changes are in main and will be included in the next release.

from aiohttp-client-cache.

These changes are available in v0.9.

from aiohttp-client-cache.

Could you provide a complete code example that reproduces the problem? I am not familiar with how Sentry interfaces with aiohttp's ClientSession.

from aiohttp-client-cache.

@JWCook
Hi
I've created a basic example that demonstrates this behavior. It sends two identical requests using, one before initializing Sentry, and a second one after initializing Sentry. The expected behavior is for the client to use the specified request class. But only the first request use CustomRequest as seen by the printed output

Dependencies:

python = "^3.11"
sentry-sdk = "^1.29.2"
aiohttp = "^3.8.5"
aiohttp-client-cache = {extras = ["all"], version = "^0.8.2"}

Code:

import asyncio

import sentry_sdk
from aiohttp import ClientRequest, ClientResponse
from aiohttp.connector import Connection
from aiohttp_client_cache import CachedSession


class CustomRequest(ClientRequest):
    async def send(self, conn: Connection) -> ClientResponse:
        print("Custom request class used!")
        return await super().send(conn)


async def send_request():
    print("Sending a request")
    async with CachedSession(request_class=CustomRequest) as session:
        await session.get('https://example.org/')
    print("Request completed\n")


if __name__ == '__main__':
    asyncio.run(send_request())

    print("Initializing Sentry and sending another request")
    sentry_sdk.init()
    asyncio.run(send_request())

stdout:

Sending a request
Custom request class used!
Request completed

Initializing Sentry and sending another request
Sending a request
Request completed

Hope this helps

from aiohttp-client-cache.

Thank you for the details!

It's a bit hard to follow exactly how sentry_sdk.init() modifies the session, but I can at least confirm that it changes the init signature to __init__(*args, **kwargs). I think we can handle this during inspection by checking for the Parameter.VAR_KEYWORD type.

from aiohttp-client-cache.