fix(docs): correct cross-language inconsistencies and add missing features#43
Open
forgetso wants to merge 3 commits into
Open
fix(docs): correct cross-language inconsistencies and add missing features#43forgetso wants to merge 3 commits into
forgetso wants to merge 3 commits into
Conversation
Replaces the one-paragraph mention of the disposable-domain blocklist with full coverage of the five-stage check chain (direct lookup, SSRF-safe validation, HTTPS redirect probe, CNAME chase, MX fallback), provider-side config (`spamEmailDomainsUrls`, scheduler cron) and the standalone `/v1/prosopo/provider/client/spam/email` endpoint with request/response examples. Also updates the evaluation-order list to reflect that the domain-list stage runs last (after the synchronous pattern rules) since it is the most expensive. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Cuts implementation detail that doesn't belong on customer-facing docs: the SSRF safety check, source-code paths, the provider-side feed URLs and cron config, and the per-step DNS-chase explanation. What's left: what the toggle does, what gets caught (including the redirect/CNAME/MX chase, named but not over-explained), the rejection status code, and the standalone endpoint contract. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…tures Cross-language fixes (en + de/es/fr/it/pt-br): - Fix broken React integration links in welcome and client-side-rendering (pointed at /angular-integration/ instead of /react-integration/) - Fix invisible-captcha server-side example using reCAPTCHA-style fields (response/remoteip/result.success) instead of Prosopo's (token/ip/verified) - Fix server-side-verification SDK example: ProsopoServer requires a pair argument; isVerified returns a VerificationResponse object, not a boolean - Fix tier naming "Pro and Enterprise" -> "Professional and Enterprise" to match the Tier enum in the portal English-only: - Add ASN as a documented access-control rule field (already supported by the rule editor and policy schema, just undocumented) - Reword traffic-filter "Providing the IP Address" to lead with what actually happens (filters always run against the session-initiation IP; passing ip overrides with a fresh lookup) instead of implying the IP field is required Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for peaceful-pothos-9e62ce ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Cross-language fixes (en + de/es/fr/it/pt-br):
welcome/andbasics/client-side-rendering.mdxpointed at/angular-integration/instead of/react-integration/(24 instances across 12 files).response,remoteip,result.success) instead of Prosopo's (token,ip,verified). MatchesApiParamsandVerificationResponseincaptcha/packages/types.new ProsopoServer(config)would crash at runtime becauseverifyProviderrequirespair.sign(...). Now passespairviagetPair(). Alsoif (await isVerified(...))treated the returned object as a boolean — fixed to unpackresult.verified. JS fetch example missingawaitonresponse.json().Tierenum in the portal (packages/portal/src/components/AccountComponents/UpgradeModal.tsx).English-only:
RuleEditorFields.tsand the policy schema (captcha/packages/user-access-policy/src/rule.ts), just undocumented.ipoverrides with a fresh lookup). Previous wording implied theipfield was required.Test plan
🤖 Generated with Claude Code