Skip to content

DEV-1815: stop apiUrl example from leaking Fliplet auth headers to third parties - #289

Merged
illia-fliplet merged 1 commit into
projects/DEV-1815from
feature/DEV-1815
Aug 19, 2026
Merged

DEV-1815: stop apiUrl example from leaking Fliplet auth headers to third parties#289
illia-fliplet merged 1 commit into
projects/DEV-1815from
feature/DEV-1815

Conversation

@veljko7

@veljko7 veljko7 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

The "Using Custom API URL" example in docs/API/core/api.md showed Fliplet.API.request() with apiUrl pointed at an external/third-party host. Fliplet.API.request() always attaches the current user's Fliplet Auth-token/Authorization headers regardless of the apiUrl override, so following that example as written leaks the user's Fliplet session credentials to whatever third-party host is set.

This PR:

  • Replaces the example with a legitimate Fliplet-owned-host use case (a regional API deployment) instead of a third-party service
  • Adds an explicit warning against pointing apiUrl at third parties, with a fetch()-based alternative for genuine third-party calls
  • Clarifies the apiUrl table row and the Authentication section to state the auth headers are attached unconditionally, even when apiUrl overrides the host

JIRA ticket

https://weboo.atlassian.net/browse/DEV-1815

Background

Found as a follow-up during review of fliplet-studio#8851 (DEV-1802), which stops the V3 AI App Action builder from generating this same dangerous apiUrl-to-third-party pattern. The builder's prompt now explicitly overrides this doc example for the AI builder, but the underlying public doc was still wrong and could mislead any developer (or another AI) reading it directly — the builder's own doc-lookup flow fetches this exact page.

Testing instructions

  1. Run cd docs && node bin/build-agent-indexes.mjs --strict — passes (confirmed locally, no generated index files changed by this wording fix).
  2. Review the rendered example and warning read correctly.

Deployment instructions

None — docs-only change, published automatically by Cloudflare Pages on merge to master (via projects/DEV-1815master).

🤖 Generated with Claude Code

…ird parties

The "Using Custom API URL" example showed Fliplet.API.request() with
apiUrl pointed at an external service. Fliplet's Auth-token/Authorization
headers are attached unconditionally regardless of apiUrl, so following
that example leaks the user's Fliplet session credentials to whatever
third-party host is set.

Replaces the example with a legitimate Fliplet-owned-host use case, adds
an explicit warning against pointing apiUrl at third parties, and points
readers to fetch() with the third party's own credentials instead.

Follow-up from fliplet-studio PR #8851 (DEV-1802) review, where this doc
example was found to contradict the fix being made to the V3 AI builder
prompt.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying fliplet-cli with  Cloudflare Pages  Cloudflare Pages

Latest commit: b4c95f2
Status: ✅  Deploy successful!
Preview URL: https://e9a37cf4.fliplet-cli.pages.dev
Branch Preview URL: https://feature-dev-1815.fliplet-cli.pages.dev

View logs

@illia-fliplet
illia-fliplet merged commit 755ed9a into projects/DEV-1815 Aug 19, 2026
3 checks passed
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