fix(log-management): update access logs formats to clever-tools v4#973
Open
hsablonniere wants to merge 1 commit into
Open
fix(log-management): update access logs formats to clever-tools v4#973hsablonniere wants to merge 1 commit into
hsablonniere wants to merge 1 commit into
Conversation
The access logs section still documented the Warp10-era CLI: the `simple` and `extended` formats no longer exist, the CLF example carried a stray `-` for the missing protocol, and the JSON sample used the old short-key payload. Align it with the current `clever accesslogs` output: `human` (default), `clf`, `json` and `json-stream`, with the v4 access log JSON shape and a CLF request line reduced to `method path`, as reported in clever-tools#909.
✅ Review app deployedCheck how your changes look like!
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
Reported in clever-tools#909: the Access logs section of
log-management.mdis out of date. It still describes the Warp10-eraclever accesslogs:simpleandextendedno longer exist (the CLI now useshuman,json,json-stream, andclf)-for the missing HTTP protocol ("GET / -")ipS,vb,bOut…) instead of the v4 access log shapeProposal
Rewrite the section to match the current CLI output:
human(default),clf,json,json-streammethod path— the protocol version is absent from the v4 payload, and a-placeholder breaks structured parsing in grok/GoAccess (see clever-tools PR #1104)ApplicationAccessLogobjectPaired with clever-tools PR #1104, which restores the
clfformat in the CLI.