Skip to content

FD-471: Replace test API keys with placeholder#531

Merged
vmangwani merged 2 commits into
mainfrom
FD-471-remove-test-api-keys
Mar 31, 2026
Merged

FD-471: Replace test API keys with placeholder#531
vmangwani merged 2 commits into
mainfrom
FD-471-remove-test-api-keys

Conversation

@vmangwani
Copy link
Copy Markdown
Contributor

Summary

  • Replaced all instances of the exposed test API key (test_0dc8d51e0acffcb1880e0f19c79b2f5b0cc) with <YOUR_TEST_API_KEY> placeholder
  • Also replaced partially-redacted variants (test_0dc8dXXXX... and test_0dc8dxxxx...) for consistency
  • Changes cover all source files (resources/, lob-api-public.yml), generated dist files, and docs

Test plan

  • Spectral lint passes (no errors found)
  • Bundle rebuilt cleanly from updated sources
  • Postman collection rebuilt from updated bundle
  • Prettier formatting applied
  • Verified no live or test API keys remain in any tracked file

🤖 Generated with Claude Code

Replaced all instances of the exposed test API key with
<YOUR_TEST_API_KEY> placeholder across source, dist, and docs files.
Ran prettier and rebuilt the bundle and Postman collection.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vmangwani vmangwani requested a review from a team as a code owner March 30, 2026 20:20
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the API documentation and resource definition files by replacing a hardcoded test API key with a generic <YOUR_TEST_API_KEY> placeholder across all curl command examples. This change ensures that users are prompted to use their own credentials when testing the API endpoints. I have no feedback to provide as there were no review comments.

dist/ is regenerated by CI so it should not be committed manually.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vmangwani vmangwani merged commit 5d35f9c into main Mar 31, 2026
5 checks passed
@vmangwani vmangwani deleted the FD-471-remove-test-api-keys branch March 31, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants