Project

General

Profile

Actions

Bug #85653

closed

Description and/or behaviour of [SYS][ipAnonymization] is wrong/misleading wrt. to sys_log

Added by Stefan P over 5 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2018-07-26
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

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:
  1. adjust the description (-> "not used by core itself for log entries, use task or anonymize API yourself if you need this")
  2. 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"

Actions #1

Updated by Daniel Siepmann about 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.

Actions #2

Updated by Gerrit Code Review about 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

Actions #3

Updated by Gerrit Code Review about 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

Actions #4

Updated by Gerrit Code Review about 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

Actions #5

Updated by Daniel Siepmann about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #6

Updated by Benni Mack about 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF