Skip to content

document cf-aig-collect-log-payload header redacting request and responses in logs #28904

Merged
ethulia merged 3 commits intoproductionfrom
minglu/aig-collect-log-payload-header
Mar 17, 2026
Merged

document cf-aig-collect-log-payload header redacting request and responses in logs #28904
ethulia merged 3 commits intoproductionfrom
minglu/aig-collect-log-payload-header

Conversation

@ethulia
Copy link
Contributor

@ethulia ethulia commented Mar 10, 2026

…request payload logging control

Summary

Screenshots (optional)

Documentation checklist

  • Is there a changelog entry (guidelines)? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to RSS feeds, the Discord, and X.
  • The change adheres to the documentation style guide.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@github-actions github-actions bot added product:ai-gateway AI Gateway: https://developers.cloudflare.com/ai-gateway/ size/s labels Mar 10, 2026
@ethulia
Copy link
Contributor Author

ethulia commented Mar 10, 2026

/bonk review this

@github-actions
Copy link
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/ai-gateway/ @abhishekkankani, @palashgo, @thebongy, @roerohan, @kathayl, @mchenco, @zeke, @superhighfives, @bfirsh, @mattrothenberg, @ethulia, @cloudflare/pcx-technical-writing

Copy link
Contributor

@ask-bonk ask-bonk bot left a comment

Choose a reason for hiding this comment

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

One suggestion posted inline. Also noted a pre-existing issue: line 37 in this file has a missing closing single quote on --header 'cf-aig-collect-log: false (should be 'cf-aig-collect-log: false'), which makes the existing curl example invalid. Consider fixing it in this PR since both examples are in the same section.

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Mar 10, 2026

Review posted on PR #28904. Here's a summary of what I found:

1 inline suggestion posted, plus 1 out-of-scope note.

  • LOW — Misleading note wording (lines 80-82): The :::note block implies both cf-aig-collect-log: false and cf-aig-collect-log-payload: false must be combined to skip the entire log, but cf-aig-collect-log: false alone already does this. Suggested rewording to clarify the precedence.

  • Out-of-scope — Pre-existing bug (line 37): The existing cf-aig-collect-log curl example is missing a closing single quote on --header 'cf-aig-collect-log: false, making it invalid. Mentioned in the review summary since it's in the same section.

The new content is otherwise well-structured, matches the existing patterns in the file, and uses correct MDX syntax.

github run

Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
@ethulia ethulia changed the title document cf-aig-collect-log-payload header redacting request and responses in logs [do not merge] document cf-aig-collect-log-payload header redacting request and responses in logs Mar 11, 2026
@ethulia ethulia changed the title [do not merge] document cf-aig-collect-log-payload header redacting request and responses in logs document cf-aig-collect-log-payload header redacting request and responses in logs Mar 17, 2026
@ethulia ethulia merged commit bc45489 into production Mar 17, 2026
11 checks passed
@ethulia ethulia deleted the minglu/aig-collect-log-payload-header branch March 17, 2026 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:ai-gateway AI Gateway: https://developers.cloudflare.com/ai-gateway/ size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.