Project

General

Profile

Actions

Feature #96147

closed

Add event / signal for redirects module

Added by Mario Rimann over 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Link Handling, Site Handling & Routing
Target version:
-
Start date:
2021-11-29
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
redirects signal event
Complexity:
Sprint Focus:

Description

We're facing a situation where a customer runs a TYPO3 10.4.x installation with several domains / subdomains configured to be handled by TYPO3's redirects module (typo3/cms-redirecs).

We added monitoring for all of those domains and subdomains, rendering the hit-count feature practically useless, since each and every check from the monitoring counts up the counter.

As a work-around for this (while keeping the monitoring in place, for reasons), we'd propose to have a signal / event in that particular part of the code, where we can add a handler that classifies requests for such redirects and does not add those that match a certain criteria to the hit-count.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #100331: List PSR-14 events in EXT:redirects documentationClosedChris Müller2023-03-28

Actions
Actions #1

Updated by Gerrit Code Review over 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #2

Updated by Gerrit Code Review over 2 years ago

Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #3

Updated by Oliver Bartsch over 2 years ago

  • Tracker changed from Bug to Feature
  • TYPO3 Version deleted (10)

Hi, thanks for creating this issue. New events usually can only be added as a "Feature", therefore I've changed the tracker accordingly.

Actions #4

Updated by Gerrit Code Review over 2 years ago

Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #5

Updated by Mario Rimann over 2 years ago

Yes, you are right, adding this Event is probably more a feature, but it solves something we see as a Bug.

So I'd say "it depends" and on the other hand: It's not that relevant, as long as it can get implemented and rolled out in one of the next v10.4.x maintenance releases and not only for the next "feature-release" (which wouldn't be v10.x then).

Anyway, THANKS for taking care and working on this issue!

Actions #6

Updated by Gerrit Code Review over 2 years ago

Patch set 4 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #7

Updated by Gerrit Code Review over 2 years ago

Patch set 5 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #8

Updated by Gerrit Code Review over 2 years ago

Patch set 6 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #9

Updated by Gerrit Code Review over 2 years ago

Patch set 7 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #10

Updated by Gerrit Code Review over 2 years ago

Patch set 8 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #11

Updated by Gerrit Code Review over 2 years ago

Patch set 9 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/72437

Actions #12

Updated by Oliver Bartsch over 2 years ago

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

Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed
Actions #14

Updated by Chris Müller about 1 year ago

  • Related to Task #100331: List PSR-14 events in EXT:redirects documentation added
Actions

Also available in: Atom PDF