Repository navigation
Conversation
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.
✅ Deploy Preview for peaceful-pothos-9e62ce ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
This is superseded by #116 and #118, which are both merged. #116 removed the auto-inject mode and 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. |
|
Everything in this PR is now on main, so closing.
One thing I deliberately didn't take: this PR describes the install as one 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. |
What changed
The Cloudflare Worker setup guide no longer offers an "auto-inject" mode or the
SITE_KEYsetting. 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.