Skip to content

Count a visitor only when they opened a page - #119

Merged
Kaliumhexacyanoferrat merged 1 commit into
mainfrom
analytics-visitors-open-pages
Oct 14, 2026
Merged

Kaliumhexacyanoferrat merged 1 commit into
mainfrom
analytics-visitors-open-pages

Conversation

@Kaliumhexacyanoferrat

Copy link
Copy Markdown
Collaborator

A visitor was counted with the first request of the day from anything that named a browser, page or not. Scanners that send a browser's user agent and guess paths (/contact, /staff, /.well-known/security.txt - 1,833 of genhttp.org's 4,562 requests were turned away) each counted as a visitor without opening anything. genhttp.org showed 1,408 visitors against 1,244 page views.

A visitor is now a person who opened a page (visit.Page and a browser), counted with the first page they opened that day - so there are never more visitors than page views.

  • VisitTally: the visitor check requires a page view
  • AVisitorIsSomebodyWhoOpenedAPage: a browser fetching a script and one calling an API are no visitors; one opening a page is (fails before the change)
  • Rule in CLAUDE.md, README, platform_guide/the analytics tool, the editor's hint and /docs, translated

What was counted before stays as it was: the hashes that tell visitors apart are never kept, so past days cannot be counted again.

🤖 Generated with Claude Code

A visitor was counted with the first request of the day from anything
that named a browser, page or not, so scanners that send a browser's
agent and guess paths (/contact, /staff, /.well-known/security.txt)
counted as visitors without opening anything. genhttp.org showed 1,408
visitors against 1,244 page views.

A visitor is now a person who opened a page, counted with the first page
they opened that day, so there are never more visitors than page views.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Kaliumhexacyanoferrat
Kaliumhexacyanoferrat merged commit 0e92220 into main Oct 14, 2026
3 checks passed
@Kaliumhexacyanoferrat
Kaliumhexacyanoferrat deleted the analytics-visitors-open-pages branch October 14, 2026 11:59
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.

1 participant