Skip to content

[SYNTH-28155] Clarify HTTP payload capture privacy - #40439

Merged
GuilhermeBorges merged 4 commits into
masterfrom
guilherme.oliveira/SYNTH-28155-payload-capture-notice
Oct 6, 2026
Merged

GuilhermeBorges merged 4 commits into
masterfrom
guilherme.oliveira/SYNTH-28155-payload-capture-notice

Conversation

@GuilhermeBorges

@GuilhermeBorges GuilhermeBorges commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Adds a notice explaining secret-variable redaction and the risk of capturing other sensitive data in HTTP payloads. Links to the existing data retention documentation.

SYNTH-28155

Merge readiness

  • Ready for merge

AI assistance

Codex assisted with drafting and validation.

Additional notes

Validation: no new Vale findings; checked the retention link and whitespace.

@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@GuilhermeBorges
GuilhermeBorges marked this pull request as ready for review October 5, 2026 19:13
@GuilhermeBorges
GuilhermeBorges requested a review from a team as a code owner October 5, 2026 19:13
@datadog-official

This comment has been minimized.

@drichards-87 drichards-87 self-assigned this Oct 5, 2026

@drichards-87 drichards-87 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Left some feedback from Docs on the PR.

Comment thread hugo/content/en/synthetics/browser_tests/_index.md Outdated
Co-authored-by: DeForest Richards <56796055+drichards-87@users.noreply.github.com>
@GuilhermeBorges
GuilhermeBorges merged commit 8ac80bb into master Oct 6, 2026
27 checks passed
@GuilhermeBorges
GuilhermeBorges deleted the guilherme.oliveira/SYNTH-28155-payload-capture-notice branch October 6, 2026 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants