Skip to content

docs: document the wipe_field_value logging filter#13374

Open
maskit wants to merge 2 commits into
apache:masterfrom
maskit:doc-wipe-field-value
Open

docs: document the wipe_field_value logging filter#13374
maskit wants to merge 2 commits into
apache:masterfrom
maskit:doc-wipe-field-value

Conversation

@maskit

@maskit maskit commented Jul 9, 2026

Copy link
Copy Markdown
Member

Documentation only

The wipe_field_value logging filter was previously mentioned only in passing, with no explanation of how it behaves or how to configure it. This adds a dedicated "Wiping Query Parameter Values" section to the logging.yaml admin guide.

What's covered

  • What it does: masks the values of matching query string parameters before the event is logged, and unlike accept/reject never drops the event.
  • The non-obvious rule: matching is against parameter names, not values — so a value that happens to equal one of the filter tokens is left untouched.
  • A worked example with a filter config and an input-URL → logged-value table.

Verification

The example config and its input/output table are taken directly from the existing log-filter autest (tests/gold_tests/logging/log-filter.replays.yaml and gold/filter-test.gold), so the documented output matches what Traffic Server actually produces rather than being written from memory.

No code changes; docs only.

Add a dedicated section describing how the wipe_field_value log
filter masks query parameter values, including that it matches
parameter names rather than values. The worked example is taken
from the existing log-filter autest so the shown output matches
what Traffic Server actually produces.
@maskit maskit added this to the 11.0.0 milestone Jul 9, 2026
@maskit maskit self-assigned this Jul 9, 2026
Copilot AI review requested due to automatic review settings July 9, 2026 15:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the Traffic Server admin logging guide by adding dedicated documentation for the wipe_field_value log filter action, explaining how it rewrites (but never drops) log events by masking matching query parameter values.

Changes:

  • Added a new “Wiping Query Parameter Values” section describing wipe_field_value behavior and configuration guidance.
  • Added an explicit note that matching applies to query parameter names (not values).
  • Added a worked example with a filter snippet and request URL → logged value table.

Comment thread doc/admin-guide/files/logging.yaml.en.rst Outdated
Comment thread doc/admin-guide/files/logging.yaml.en.rst Outdated
Copilot AI review requested due to automatic review settings July 9, 2026 21:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread doc/admin-guide/files/logging.yaml.en.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants