Skip to content

docs: re-measure latency with Safe Browsing enabled - #1

Merged
itsF4LCON merged 1 commit into
mainfrom
docs/latency-with-safe-browsing
Oct 8, 2026
Merged

itsF4LCON merged 1 commit into
mainfrom
docs/latency-with-safe-browsing

Conversation

@itsF4LCON

Copy link
Copy Markdown
Owner

Safe Browsing (SAFE_BROWSING_API_KEY) is now set in production, so I re-ran the latency suite against the deployed Worker. Google's phishing and malware test pages now come back blocked as "Listed by Google Safe Browsing".

Re-measured 2026-10-08 from Belgium (Cloudflare CDG). Median time to first byte, network included:

request before (no Safe Browsing) now
network baseline (401) 38 ms 41 ms
trusted host 90 ms 70 ms
cached URL, extension 99 ms 100 ms
cached URL, API key 136 ms 123 ms
new URL, known domain 469 ms 450 ms
new URL, new domain (RDAP) 673 ms 706 ms

No measurable cost from Safe Browsing. It runs in parallel with the cache read and RDAP, and the differences are within run-to-run noise.

Model false positives: in this run the model blocked citi.com/aps-timing2, reading my made-up test path as a "suspicious subdomain". With nya5.com from the first run, that makes 2 of 50 new URLs on legitimate domains. The README now says so.

README only, no code changes. The site's claims in itsF4LCON/xivlabs#4 (about 100 ms cached, about half a second new) still hold.

Safe Browsing is now configured in production. New URLs are no slower
within run-to-run noise (450 ms and 706 ms medians). The model blocked
one more legitimate URL in this run, now 2 of 50 overall.
Copilot AI balanced review requested due to automatic review settings October 8, 2026 19:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@itsF4LCON
itsF4LCON merged commit 9f680d3 into main Oct 8, 2026
1 check 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.

2 participants