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

parse nginx log_format #4

Open
Canop opened this issue Mar 3, 2021 · 1 comment
Open

parse nginx log_format #4

Canop opened this issue Mar 3, 2021 · 1 comment

Comments

@Canop
Copy link
Owner

Canop commented Mar 3, 2021

Right now rhit assumes the log format is the default one, which is quite common.

Among the reasons I had not to interpret the configured log format:

  • it's painful to write an efficient enough variable parser
  • changing the configuration means the past isn't readable anymore, so it can't be perfect
  • the log_format variable lets you define unparsable and ambiguous log lines
  • you may define formats missing essential fields

So, as long as it looks like I can have something working for 99% of users, using a few tricks and dynamic recognitions, I won't implement this issue.

Nevertheless, I'm not sure not parsing the log_format will always be OK, hence this issue.

@im-n1
Copy link

im-n1 commented Feb 21, 2023

Okay, fits to my case.

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

No branches or pull requests

2 participants