Skip to content

Traefik log support #320

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

Merged
merged 5 commits into from
Apr 11, 2025
Merged

Traefik log support #320

merged 5 commits into from
Apr 11, 2025

Conversation

albank
Copy link
Contributor

@albank albank commented Sep 20, 2021

Description:

Please include a description of this change and which issue it fixes. If no issue exists yet please include context and what problem it solves.

Review

@sgiehl sgiehl added this to the Current sprint milestone Sep 20, 2021
@innocraft-automation innocraft-automation removed this from the Current sprint milestone Jan 24, 2023
sgiehl and others added 3 commits September 6, 2023 09:20
* Update test action

* fix older versions

* fix action
* Add Traefik JSON log format

* Add Traefik JSON log format

* Add Traefik json log support.

What was done:
1. Expanded import_logs.py to auto recognize and import Traefik json log files.
2. Modified the NGINX json support, made the autodetect function more specific. It was to generic. All json formatted logs where seen as NGINX logs.
3. Added tests for Traefik log importing.

* Fixed failing tests and refactored get_all method

What was done:
* The get_all method was returning the raw json without any key mapping. Fixed that. This meant also changing the names in test_main.yml
* changed log file form CLF to LF

---------

Co-authored-by: David Patiashvili <stratagem.david@gmail.com>
@sgiehl sgiehl changed the base branch from 4.x-dev to 5.x-dev April 11, 2025 08:30
@sgiehl sgiehl merged commit bedcde8 into matomo-org:5.x-dev Apr 11, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants