🌐 AI搜索 & 代理 主页
Skip to content

Conversation

@lyrixx
Copy link
Member

@lyrixx lyrixx commented Sep 13, 2022

Q A
Branch? 6.2
Bug fix? no
New feature? yes
Deprecations? no
Tickets Fix #45599
License MIT
Doc PR

@lyrixx
Copy link
Member Author

lyrixx commented Sep 13, 2022

I guess I must edit a CHANGELOG.md but I don't understand how it work.

Is it src/Symfony/Contracts/CHANGELOG.md? what version should I pick?

@nicolas-grekas
Copy link
Member

Tests fail, we need to override some methods from the base class.

@nicolas-grekas
Copy link
Member

Should we do this only when the contracts are used? Aka when using the PSR-6 interface, still throw?

@upyx
Copy link
Contributor

upyx commented Sep 18, 2022

I've described why it would be better to still forbid semicolons in cache keys in the #45599.

@nicolas-grekas
Copy link
Member

I'm closing here because this was a poc and it showed we need more discussion - also because the implementation is likely not the one we'll want.

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.

[Cache] Allow colon in key key (with redis at least)

4 participants