Skip to content

Conversation

tvernum
Copy link
Contributor

@tvernum tvernum commented Sep 26, 2025

Backports the following commits to 8.19:

In audit logs we redact certain fields from the body of rest requests.

This commit changes the way we redact fields in the reindex request.
Previously the only form of redaction we supported was total removal
of fields, however that can be problematic when an admin wants to know
whether a field was supplied or not.

Here we change the way we redact requests for reindexing to replace
fields with `::es-redacted::` instead of removing them.
@tvernum tvernum added :Security/Audit X-Pack Audit logging >enhancement auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Security Meta label for security team labels Sep 26, 2025
@tvernum
Copy link
Contributor Author

tvernum commented Sep 26, 2025

Merging due to know problem with packaging tests on Ubuntu 24

@tvernum tvernum merged commit 94e64b1 into elastic:8.19 Sep 26, 2025
27 of 30 checks passed
@tvernum tvernum deleted the backport/8.19/pr-135414 branch September 26, 2025 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >enhancement :Security/Audit X-Pack Audit logging Team:Security Meta label for security team v8.19.5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants