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

Add Caddy json log support #366

Closed
wants to merge 142 commits into from
Closed

Conversation

simon04
Copy link
Contributor

@simon04 simon04 commented Dec 24, 2023

geographika and others added 30 commits June 26, 2017 12:57
this allows multiple log_writers[] in the config.ini.php
mwithheld and others added 28 commits March 4, 2021 15:42
Some formats include a colon in the timezone, e.g. -08:00
Allows to reports downloaded signature files for what they are: downloads!
Reports signature files as downloads
…t API. The response was detected in byte. Adding a decoder depending on the response.
Add Gandi VirtualHost log format
…tracking

Fix disabling of queued tracking with bulk requests.
Mention Matomo for WordPress isn't compatible
Use a GitHub action instead of Travis CI
Set referrer to empty string if it doesn't exist
* 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>
@michalkleiner
Copy link
Contributor

Closing as superseded by #367.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.