Skip to content

Add missing . to jq log parsing example

Greg Myers requested to merge docs-jq-log-parse-example-fix into master

What does this MR do?

Fixes the example here by adding . before status.

-- jq 'select(status >= 500)' <FILE>
++ jq 'select(.status >= 500)' <FILE>

Screenshots or Screencasts (strongly suggested)

Before

$ greg@gitlab:[~]:) jq 'select(status >= 500)' production_json.log

jq: error: status/0 is not defined at <top-level>, line 1:
select(status >= 500)       
jq: 1 compile error

After

$ greg@gitlab:[~]:) jq 'select(.status >= 500)' production_json.log

{
  "method": "GET",
  "path": "/user/baritone/-/graphs/918f009e8a02fcfa1edbf0cda360429da6f62083",
  "format": "html",
  "controller": "Projects::GraphsController",
  "action": "show",
  "status": 500,
  "time": "2021-07-29T00:40:07.301Z",
  "params": [
    {
      "key": "namespace_id",
      "value": "user"
    },
    {
      "key": "project_id",
      "value": "baritone"
    },
    {
      "key": "id",
      "value": "918f009e8a02fcfa1edbf0cda360429da6f62083"
    }
    <truncated>
  ],
}

How to setup and validate locally (strongly suggested)

  1. Install jq
  2. Download production_json.log
  3. Reproduce problem jq 'select(status >= 500)' production_json.log
  4. Verify that adding . before status fixes the problem jq 'select(.status >= 500)' production_json.log

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Greg Myers

Merge request reports

Loading