Skip to content

Conversation

@xabbuh
Copy link
Member

@xabbuh xabbuh commented Nov 3, 2025

Q A
Branch? 6.4
Bug fix? yes
New feature? no
Deprecations? no
Issues
License MIT

@carsonbot carsonbot added this to the 6.4 milestone Nov 3, 2025
@carsonbot carsonbot changed the title [Cache] compatibility with ext-redis 6.3 [Cache] compatibility with ext-redis 6.3 Nov 3, 2025
@xabbuh
Copy link
Member Author

xabbuh commented Nov 3, 2025

preparation for the upcoming 6.3 release, let's wait for the final tag before merging

@ro0NL
Copy link
Contributor

ro0NL commented Nov 3, 2025

im wondering if can we can rely on magic __call for the wrapper, and skip this maintenance/coupling

the alternative to me is listing supported ext-redis in composer.json

@xabbuh
Copy link
Member Author

xabbuh commented Nov 3, 2025

__call() does not help here as it will not be called for known methods

@ro0NL
Copy link
Contributor

ro0NL commented Nov 3, 2025

i see 👍 Redis6Proxy extends \Redis

it feels slippery slope implementation-wise, but on our side we're pinned to ext-redis-6.2.0 now :')

@nicolas-grekas
Copy link
Member

@ro0NL you're looking for phpredis/phpredis#2753

@nicolas-grekas
Copy link
Member

Thank you @xabbuh.

@nicolas-grekas nicolas-grekas merged commit ec0cf2b into symfony:6.4 Nov 12, 2025
8 of 11 checks passed
@xabbuh xabbuh deleted the redis-6.3 branch November 12, 2025 13:18
This was referenced Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants