Bug #85653
closedDescription and/or behaviour of [SYS][ipAnonymization] is wrong/misleading wrt. to sys_log
100%
Description
The description of this global settings is
Configures if and how IP addresses stored via TYPO3''s API should be anonymized ("masked") with a zero-numbered replacement.
I totally expected this to mean that newly created sys_log entries (and other IP logs if the core does this) are automatically anonymized if set to 1 or 2. But instead this option is currently only used by the new IpAnonymizationTask and Indexed Search. I expect that the TYPO3 core itself has IP addressed "stored via TYPO3''s API".
I see two options here:- adjust the description (-> "not used by core itself for log entries, use task or anonymize API yourself if you need this")
- make the core actually anonymize IP addresses upon persisting sys_log entries depending on this setting
I prefer the second option, because then this setting would be of an actual use without having to setup the task to anonymize afterwards.
Side note: also note the wrong double apostrophe in the string "TYPO3''s API"
Updated by Daniel Siepmann over 4 years ago
- Status changed from New to Accepted
- Assignee set to Daniel Siepmann
For now I'll adjust the description. Thanks for pointing out this misleading information.
Updated by Gerrit Code Review over 4 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63753
Updated by Gerrit Code Review over 4 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63753
Updated by Gerrit Code Review over 4 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63893
Updated by Daniel Siepmann over 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 295873fcb5b382e01f7e56c97df27c94a4c6f5a9.