-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Rewrite AccessLogformatter with parser library #2967
Copy link
Copy link
Closed as not planned
Labels
enhancementFeature requests. Not bugs or questions.Feature requests. Not bugs or questions.stalestalebot believes this issue/PR has not been touched recentlystalebot believes this issue/PR has not been touched recentlytech debt
Description
We're doing a lot of manual crufty stuff with substrings and parentheses matching to parse out the require format strings. We should probably switch to some formal grammar for the access log format strings (e.g. EBNF) and use a parser library or generator (e.g. yacc). This issue will track improvements here.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementFeature requests. Not bugs or questions.Feature requests. Not bugs or questions.stalestalebot believes this issue/PR has not been touched recentlystalebot believes this issue/PR has not been touched recentlytech debt