Skip to content

Fix Symfony profiler wrapping for tag-aware cache pools#59

Merged
jstojiljkovic merged 1 commit into
tracewayapp:masterfrom
d-mitrofanov-v:fix/traceable-tag-aware-cache-pool
Jul 7, 2026
Merged

Fix Symfony profiler wrapping for tag-aware cache pools#59
jstojiljkovic merged 1 commit into
tracewayapp:masterfrom
d-mitrofanov-v:fix/traceable-tag-aware-cache-pool

Conversation

@d-mitrofanov-v

Copy link
Copy Markdown
Contributor

Hi, everyone! Got an error in my tests, when added bundle to my new project. The problem is that Symfony’s CacheCollectorPass checks for TagAwareAdapterInterface when deciding to wrap a cache pool with TraceableTagAwareAdapter or TraceableAdapter, but TraceableTagAwareCachePool doesn't implement TagAwareAdapterInterface.

Here is the error that i had:

{
    "error": {
        "title": "Server error",
        "status": 500,
        "errorCode": null,
        "detail": "App\\User\\Service\\UserService::__construct(): Argument #3 ($appCacheUser) must be of type Symfony\\Contracts\\Cache\\TagAwareCacheInterface, Symfony\\Component\\Cache\\Adapter\\TraceableAdapter given, called in /var/www/var/cache/test/ContainerAbJwkYx/App_KernelTestDebugContainer.php on line 7847"
    }
}

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@jstojiljkovic

Copy link
Copy Markdown
Collaborator

Thanks for the contribution, merged will be released soon. 🙇

@jstojiljkovic
jstojiljkovic merged commit 62c898f into tracewayapp:master Jul 7, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants