NotificationsYou must be signed in to change notification settings
Fork9.6k
Star30.4k

[Uid] Add InvalidUidException Context#60557

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

rela589n wants to merge2 commits intosymfony:7.4

base:7.4

Choose a base branch

fromrela589n:invalid-uid-exception

Open

[Uid] Add InvalidUidException Context#60557

rela589n wants to merge2 commits intosymfony:7.4fromrela589n:invalid-uid-exception

Conversation

Copy link

Contributor

rela589n commentedMay 27, 2025

Q	A
Branch?	7.4
Bug fix?	no
New feature?	yes (minor)
Deprecations?	no
Issues	#60328
License	MIT

Considering the main purpose of the exceptions - to be logged, it is best to have their context normalized, so that it can be easily processed by special tools (like Graylog, Logstash, etc).

This PR adds context toInvalidUidException so that its value will appear in logs, and also it adds PSR3-interpolation placeholders for uid value where such were absent.