Skip to content

Remove index audit output type #37707

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.

Sign up for GitHub

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

Merged
merged 14 commits into from
Jan 24, 2019
Merged

Remove index audit output type #37707

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
36fc57f
WIP
albertzaharovits Jan 22, 2019
62547c5
All deletes are in!
albertzaharovits Jan 22, 2019
71dc58c
The docs
albertzaharovits Jan 22, 2019
2f314c3
Migrate note
albertzaharovits Jan 22, 2019
edbf2c7
Apply suggestions from code Jay's review
jaymode Jan 22, 2019
c9b6d69
Update x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/s…
jaymode Jan 22, 2019
87b7d88
Address review
albertzaharovits Jan 22, 2019
8219937
Feedback
albertzaharovits Jan 22, 2019
309f215
Mooore reviews!
albertzaharovits Jan 22, 2019
a512fb8
Merge branch 'master' into remove_audit_index
albertzaharovits Jan 23, 2019
a106500
Fix AuditTrailSettingsUpdateTests
albertzaharovits Jan 23, 2019
4cb3a89
remove audit.outputs from sql/qa/security
albertzaharovits Jan 23, 2019
865ff6e
Remove IndexAuditUpgradeIT from qa/rolling-upgrade
albertzaharovits Jan 23, 2019
d4a0f85
Merge branch 'master' into remove_audit_index
albertzaharovits Jan 24, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions docs/reference/settings/audit-settings.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,7 @@ file. For more information, see
`xpack.security.audit.enabled`::
Set to `true` to enable auditing on the node. The default value is `false`.
This puts the auditing events in a dedicated file named `<clustername>_audit.log`
on each node.
+
For more information, see <<configuring-logging-levels>>.
on each node. For more information, see <<configuring-logging-levels>>.

[[event-audit-settings]]
==== Audited Event Settings
Expand Down
6 changes: 3 additions & 3 deletions x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/Security.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -376,9 +376,9 @@ Collection<Object> createComponents(Client client, ThreadPool threadPool, Cluste
components.add(securityContext.get());

// audit trail service construction
final List<AuditTrail> auditTrails = XPackSettings.AUDIT_ENABLED.get(settings) ?
Collections.singletonList(new LoggingAuditTrail(settings, clusterService, threadPool)) :
Collections.emptyList();
final List<AuditTrail> auditTrails = XPackSettings.AUDIT_ENABLED.get(settings)
? Collections.singletonList(new LoggingAuditTrail(settings, clusterService, threadPool))
: Collections.emptyList();
if (XPackSettings.AUDIT_ENABLED.get(settings)) {
// the only audit trail is the logfile
auditTrails.add(new LoggingAuditTrail(settings, clusterService, threadPool));
Expand Down