Skip to content

feat(roles/apache_httpd): add csv log formats (and tsv equivalents) - #391

Open
ebuerki-lf wants to merge 4 commits into
mainfrom
feat/apache_httpd_csvlog
Open

ebuerki-lf wants to merge 4 commits into
mainfrom
feat/apache_httpd_csvlog

Conversation

@ebuerki-lf

Copy link
Copy Markdown
Contributor

No description provided.

… csvsiem" and their tsv equivalents "tsvio tsvextensive tsvsiem"

@NavidSassan NavidSassan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didnt have a closer look at the formats themselves yet, most comments are about the style

@@ -1,5 +1,5 @@
# {{ ansible_managed }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

commit message should start with feat(roles/apache_httpd)

Comment thread roles/apache_httpd/templates/etc/httpd/conf-available/logio.conf.j2 Outdated

LogFormat "%h %{X-Forwarded-For}i %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\" %I %O" linuxfabrikio

# io: based on linuxfabrikio

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# io: based on linuxfabrikio
# csvio, tsvio: based on linuxfabrikio logformat

Comment thread roles/apache_httpd/templates/etc/httpd/conf-available/logio.conf.j2 Outdated
Comment thread roles/apache_httpd/templates/etc/httpd/conf-available/logio.conf.j2 Outdated
Comment thread roles/apache_httpd/templates/etc/httpd/conf-available/logio.conf.j2 Outdated
…nt for csv/tsv log formats. log cloudflare headers CF-Ray CF-Connecting-IP in all csv/tsv formats
@ebuerki-lf ebuerki-lf changed the title feat(apache_httpd): add csv log formats (and tsv equivalents) feat(roles/apache_httpd): add csv log formats (and tsv equivalents) Sep 18, 2026
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.

2 participants