Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zu ignorierende Pfade & Zu ignorierende REGEX scheint nicht richtig zu funktionieren #120

Open
iceman-fx opened this issue Aug 14, 2024 · 3 comments
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed

Comments

@iceman-fx
Copy link

iceman-fx commented Aug 14, 2024

Ich habe aktuell das Problem, dass ein Formularaufruf ca. 500000 Einträge in 6 Monaten in der DB ablegt, was den Aufruf des Addons massiv verlangsamt. Grund sind irgendwelche Trolle, die versuchen Schindluder mit dem Formular zu produzieren (= massig Seitenaufrufe von unterschiedlichen IPs aus).

Der Versuch, diesen Aufruf gar nicht erst zu loggen, schlägt jedoch fehl.
Egal was ich in den Einstellungsfeldern definiere, es wird weiterhin geloggt.

Zu ignorierende Pfade:
www.domain.de/service/kurszertifikat-abrufen

und/oder

Zu ignorierende REGEX:
^www\.domain\.de\/service\/kurszertifikat-abrufen

Was mache ich falsch oder liegt da ein Bug vor?
Beide Varianten sollten doch einen Pfad, welcher mit www.domain.de/service/kurszertifikat-abrufen beginnt nicht loggen?

@iceman-fx
Copy link
Author

Ist dies hier die Prozedur zum loggen des Aufrufes?
Wenn ja, dann wird hier doch gar nix bzgl. Pfade/Regex ignorieren geprüft?

        if (rex::getRequest()->getRequestUri() != "/favicon.ico") {

            if ($response_code == rex_response::HTTP_OK || !$addon->getConfig("statistics_rec_onlyok", false)) {
                // visitduration, number pages visited, last visited page
                $sql = rex_sql::factory();
                $sql->setQuery("INSERT INTO " . rex::getTable('pagestats_sessionstats') . " (token, lastpage, lastvisit, visitduration, pagecount) VALUES (:token, :lastpage, NOW(), 0, 1) ON DUPLICATE KEY UPDATE lastpage = VALUES(lastpage), visitduration = visitduration + (NOW() - lastvisit), lastvisit = NOW(), pagecount = pagecount + 1", [":token" => $token, ":lastpage" => $url]);
            }
        }

@AndiLeni
Copy link
Owner

AndiLeni commented Sep 3, 2024

Aktuell werden für die Sessionstats die Einstellungen "Zu ignorierende Pfade & Zu ignorierende REGEX" nicht angewendet.

@AndiLeni AndiLeni added the enhancement New feature or request label Sep 3, 2024
@alxndr-w
Copy link
Collaborator

Für das ursprüngliche Anliegen würde ich empfehlen, eine Firewall zu verwenden und Filter, die Bots nach bestimmten Aktivitäten komplett ausschließen.

Generell sind für eine Datenbank 500.000 Datensätze 'n Witz - jedoch vielleicht für die aktuelle Umsetzung des statistics-Addons ein Problem, wenn diese vorgehalten und nicht selektiert werden.

@alxndr-w alxndr-w added bug Something isn't working help wanted Extra attention is needed labels Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants