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

Teach elasticsearch/audit fileset to parse out some more fields #10137

Merged
merged 8 commits into from
Jan 18, 2019
Merged

Teach elasticsearch/audit fileset to parse out some more fields #10137

merged 8 commits into from
Jan 18, 2019

Conversation

ycombinator
Copy link
Contributor

@ycombinator ycombinator commented Jan 17, 2019

Resolves #10134.

This PR teaches the elasticsearch/audit fileset to parse out a few more fields, viz:

  • elasticsearch.audit.realm,
  • elasticsearch.audit.roles
  • elasticsearch.audit.indices

It also teaches the fileset to parse elasticsearch.audit.action values that themselves contain [ and ] delimiters around sub-actions, e.g. action=[indices:data/read/search[free_context]].

@ycombinator ycombinator added enhancement in progress Pull request is currently in progress. Filebeat Filebeat needs_backport PR is waiting to be backported to other branches. v7.0.0 Feature:Stack Monitoring v6.7.0 labels Jan 17, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/stack-monitoring

@ycombinator ycombinator requested review from a team as code owners January 17, 2019 00:25
@ycombinator ycombinator changed the title [WIP] Teach elasticsearch/audit fileset to parse out some more fields Teach elasticsearch/audit fileset to parse out some more fields Jan 17, 2019
@ycombinator ycombinator added review and removed in progress Pull request is currently in progress. labels Jan 17, 2019
Copy link
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

LGTM. Will need a rebase.

"field": "message",
"pattern_definitions": {
"ES_TIMESTAMP": "\\[%{TIMESTAMP_ISO8601:elasticsearch.audit.timestamp}\\]",
"ES_NODE_NAME": "(\\[%{DATA:elasticsearch.node.name}\\])?",
Copy link
Contributor

Choose a reason for hiding this comment

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

I quite like what you did here with the pattern definitions. It makes the grok pattern more readable.

@ycombinator ycombinator merged commit 210460e into elastic:master Jan 18, 2019
@ycombinator ycombinator removed the needs_backport PR is waiting to be backported to other branches. label Jan 23, 2019
ycombinator added a commit that referenced this pull request Jan 24, 2019
… out some more fields (#10271)

* Teach elasticsearch/audit fileset to parse out some more fields (#10137)

* Be more lenient in parsing node name

* Parse out elasticsearch.audit.realm

* Adding CHANGELOG entry

* Parse out elasticsearch.audit.roles

* Parse out elasticsearch.audit.indices

* Parse out optional sub-action

* Regenerating generated files

* Regenerating generated files

(cherry picked from commit 210460e)

* Regenerating generated files

* Fixing types from array => keyword
@ycombinator ycombinator added v6.6.1 needs_backport PR is waiting to be backported to other branches. labels Jan 24, 2019
@ycombinator ycombinator added needs_backport PR is waiting to be backported to other branches. and removed needs_backport PR is waiting to be backported to other branches. labels Jan 29, 2019
@ycombinator ycombinator removed the needs_backport PR is waiting to be backported to other branches. label Feb 1, 2019
ycombinator added a commit that referenced this pull request Feb 1, 2019
… out some more fields (#10484)

Cherry-pick of PR #10137 to 6.6 branch. Original message: 

Resolves #10134.

This PR teaches the `elasticsearch/audit` fileset to parse out a few more fields, viz:

* `elasticsearch.audit.realm`,
* `elasticsearch.audit.roles`
* `elasticsearch.audit.indices`

It also teaches the fileset to parse `elasticsearch.audit.action` values that themselves contain `[` and `]` delimiters around sub-actions, e.g. `action=[indices:data/read/search[free_context]]`.
@ycombinator ycombinator deleted the fb-es-audit-more-fields branch December 25, 2019 11:15
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…o parse out some more fields (elastic#10484)

Cherry-pick of PR elastic#10137 to 6.6 branch. Original message: 

Resolves elastic#10134.

This PR teaches the `elasticsearch/audit` fileset to parse out a few more fields, viz:

* `elasticsearch.audit.realm`,
* `elasticsearch.audit.roles`
* `elasticsearch.audit.indices`

It also teaches the fileset to parse `elasticsearch.audit.action` values that themselves contain `[` and `]` delimiters around sub-actions, e.g. `action=[indices:data/read/search[free_context]]`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants