Skip to content

docs: Cloudflare Worker setup always uses the script snippet - #115

Closed
goastler wants to merge 1 commit into
mainfrom
docs-worker-no-script-injection
Closed

goastler wants to merge 1 commit into
mainfrom
docs-worker-no-script-injection

Conversation

@goastler

Copy link
Copy Markdown
Member

What changed

The Cloudflare Worker setup guide no longer offers an "auto-inject" mode or the SITE_KEY setting. It tells customers to add the Protect script snippet to their own pages, and explains why the Worker doesn't do it for them.

Also fixes a troubleshooting link that pointed at a heading which no longer existed.

Why

The Worker stops editing pages in https://github.com/prosopo/Protect/pull/1345

Tests

Docs only. Checked the headings and links on the two edited pages by hand.

The Worker no longer injects the Protect script, so the guide drops the
auto-inject mode and SITE_KEY and shows the snippet customers add. Also
fixes a dead troubleshooting link to that step.
@goastler goastler self-assigned this Oct 16, 2026
@netlify

netlify Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for peaceful-pothos-9e62ce ready!

Name Link
🔨 Latest commit d6af7b5
🔍 Latest deploy log https://app.netlify.com/projects/peaceful-pothos-9e62ce/deploys/6ad1dbd7ca48c23df55ac4ee
😎 Deploy Preview https://deploy-preview-115--peaceful-pothos-9e62ce.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@forgetso

Copy link
Copy Markdown
Member

This is superseded by #116 and #118, which are both merged.

#116 removed the auto-inject mode and SITE_KEY from the guide, and #118 did the troubleshooting link fix this PR also carries (the anchor now points at #2a-add-the-snippet). I checked every cloudflare-worker#… anchor used anywhere in the site after that change and they all resolve.

The diff here is against the pre-#116 page, so it would conflict wholesale rather than rebase cleanly.

One thing this PR does better and I'd happily take: the framing. "Why the worker doesn't add the script for you" as a heading, plus the note that Lambda@Edge works the same way, is more useful to a customer than the single sentence that's on main now. Want me to port that bit, or would you rather take it yourself? Either way this branch can close.

@forgetso

Copy link
Copy Markdown
Member

Everything in this PR is now on main, so closing.

One thing I deliberately didn't take: this PR describes the install as one <script> tag. There are two — the window.prosopo_config block and the bundle — and the paragraph under the snippet explains why both are needed and what breaks without the first, so main keeps two.

Also worth knowing: the Workers Cache warning in #114 is in too. I rebased it onto main (it conflicted with the step renumbering) and it merged.

@forgetso forgetso closed this Oct 20, 2026
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.

2 participants