Skip to content

docs: correct withheld-credentials log entry wording (VAP15-79) - #1304

Merged
jeffrey-hoekman-vapi merged 2 commits into
mainfrom
jeffhoekman/vap15-79-correct-the-confirming-a-withheld-send-in-your-logs-section
Oct 12, 2026
Merged

docs: correct withheld-credentials log entry wording (VAP15-79)#1304
jeffrey-hoekman-vapi merged 2 commits into
mainfrom
jeffhoekman/vap15-79-correct-the-confirming-a-withheld-send-in-your-logs-section

Conversation

@jeffrey-hoekman-vapi

@jeffrey-hoekman-vapi jeffrey-hoekman-vapi commented Oct 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Fixes VAP15-79. Corrects the "Confirming a withheld send in your logs" section added in VAP15-53 (fern/server-url/server-authentication.mdx).

  • Replace "once per message type, not once per send": the announce path has no dedupe, so the entry is written on each send of a non-suppressed message type and can repeat within a call.
  • Tell readers to search the call log for the text credentials withheld instead of implying a webhook category filter. The UserLoggerV2 copy (VAPSEC-756) is filed under serverMessage, which the dashboard Category filter doesn't offer. The V1 copy has the same text, so this holds whether or not VAPSEC-756 has deployed.
  • Put the example log line on one line to match the real entry, so it can be copied into a search.
  • Unchanged: suppressed-types list and the "a missing entry does not mean credentials were sent" warning.
  • Intentionally not named on the page: the serverMessage category / serverMessage.credentialsWithheld event, until VapiAI/vapi#22109 (VAPICS-1494) lands.

Merge gate: before the VAPSEC-564 enable on 2026-10-14.

Testing Steps

  • fern check: 0 errors (12 warnings, all pre-existing OpenAPI discriminator warnings)
  • Log line text matches staging output: Assistant credentials withheld from end-of-call-report: this server url came from an assistant this request supplied, not from your org configuration
  • Run the app locally using fern docs dev or navigate to preview deployment
  • Ensure that the changed pages and code snippets work

🤖 Generated with Claude Code

@lightsage-app

lightsage-app Bot commented Oct 9, 2026 •
edited
Loading

Copy link
Copy Markdown

Lightsage docs evals

Lightsage could not queue docs evals for this PR.

Docs URL: https://vapi-preview-01a131cc-a62c-72bc-b1f3-cd3f53ced728.docs.buildwithfern.com
Commit: f2cc988
Reason: No custom evals are selected for GitHub PR evals. Open Lightsage > Custom Evals and enable the GitHub checkbox for at least one custom eval.

@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

🌿 Preview your docs: https://vapi-preview-01a120e5-3d3c-711f-89a4-a491a2fe1115.docs.buildwithfern.com

@jeffrey-hoekman-vapi @claude
docs: correct withheld-credentials log entry wording (VAP15-79)
585c2d5 
- The entry is written on each send of a non-suppressed message type,
  so it can repeat within a call (no dedupe in the announce path).
- Tell readers to search the call log for "credentials withheld"
  instead of implying a webhook category filter.
- Put the example log line on one line to match the real entry.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jeffrey-hoekman-vapi
jeffrey-hoekman-vapi force-pushed the jeffhoekman/vap15-79-correct-the-confirming-a-withheld-send-in-your-logs-section branch from 93483b6 to 585c2d5 Compare October 9, 2026 15:52
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

🌿 Preview your docs: https://vapi-preview-01a1215e-e3bc-756b-8bb1-bc07587bade1.docs.buildwithfern.com

stephenvapiai
stephenvapiai reviewed Oct 12, 2026
View reviewed changes

@stephenvapiai stephenvapiai 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.

Two small editorial suggestions: name the Logs tab and Webhook category in the search instruction, and split the recurrence sentence for readability.

Comment thread fern/server-url/server-authentication.mdx Outdated
Comment thread fern/server-url/server-authentication.mdx Outdated
@jeffrey-hoekman-vapi @claude
docs: apply review suggestions on withheld-credentials log section (V…
f2cc988 
…AP15-79)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

🌿 Preview your docs: https://vapi-preview-01a131cc-a62c-72bc-b1f3-cd3f53ced728.docs.buildwithfern.com

@jeffrey-hoekman-vapi
jeffrey-hoekman-vapi merged commit 461c3db into main Oct 12, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stephenvapiai stephenvapiai stephenvapiai approved these changes
@amanda-vapi amanda-vapi Awaiting requested review from amanda-vapi

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jeffrey-hoekman-vapi @stephenvapiai