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

DOCS Audit event attributes in new format #35510

Merged

Conversation

albertzaharovits
Copy link
Contributor

This is the spring cleaning in the audit event attributes docs.
A lot has changed and the docs have been left behind.
Some of the documented updates:

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security


The following list shows attributes that are common to all audit events.
Their names and values are analogous to those in the deprecated `logfile` or
`index` output formats. However, it is expected that the formats will evolve
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it correct that the index output format is deprecated? If not, maybe change this phrase to "... the index or deprecated logfile output format..."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I intended to keep it vague.
It is not technically deprecated yet. But I very much expect it to be in 6.x . We first need to have the filebeat parse the new logfile format . After that, deprecate it and write a blog post.

Is it really ambiguous as I intended it to be? If it implies that it is deprecated now, then it is wrong and I should correct it.

Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

I have one question, otherwise LGTM. I verified that it builds successfully.

@albertzaharovits
Copy link
Contributor Author

Thank you @lcawl !

@albertzaharovits
Copy link
Contributor Author

run gradle build tests

1 similar comment
@albertzaharovits
Copy link
Contributor Author

run gradle build tests

@albertzaharovits
Copy link
Contributor Author

run gradle build tests 1
and
run gradle build tests 2

@albertzaharovits albertzaharovits merged commit 36819f7 into elastic:master Nov 27, 2018
@albertzaharovits albertzaharovits deleted the docs_audit_event_types branch November 27, 2018 23:24
albertzaharovits added a commit that referenced this pull request Nov 28, 2018
Accounts for the `Structured Audit Entries` in the format
documentation.
albertzaharovits added a commit that referenced this pull request Nov 28, 2018
Accounts for the `Structured Audit Entries` in the format
documentation.
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Nov 28, 2018
* master:
  DOCS Audit event attributes in new format (elastic#35510)
  Scripting: Actually add joda time back to whitelist (elastic#35965)
  [DOCS] fix HLRC ILM doc misreferenced tag
  Add realm information for Authenticate API (elastic#35648)
  [ILM] add HLRC docs to remove-policy-from-index (elastic#35759)
  [Rollup] Update serialization version after backport
  [Rollup] Add more diagnostic stats to job (elastic#35471)
  Build: Fix gradle build for Mac OS (elastic#35968)
  Adds deprecation logging to ScriptDocValues#getValues. (elastic#34279)
  [Monitoring] Make Exporters Async (elastic#35765)
  [ILM] reduce time restriction on IndexLifecycleExplainResponse (elastic#35954)
  Remove use of AbstractComponent in xpack (elastic#35394)
  Deprecate types in search and multi search templates. (elastic#35669)
  Remove fromXContent from IndexUpgradeInfoResponse (elastic#35934)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes :Security/Audit X-Pack Audit logging v6.5.1 v6.6.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants