Skip to content

[Radar] Add bot vs. human traffic glossary entry#31214

Open
cdeath wants to merge 1 commit into
productionfrom
cdeath/radar-glossary-bot-vs-human
Open

[Radar] Add bot vs. human traffic glossary entry#31214
cdeath wants to merge 1 commit into
productionfrom
cdeath/radar-glossary-bot-vs-human

Conversation

@cdeath
Copy link
Copy Markdown
Collaborator

@cdeath cdeath commented Jun 3, 2026

Summary

Adds a "Bot vs. human traffic" glossary entry to the Radar glossary page. This clarifies how Radar classifies traffic as bot or human (based on bot scores) and documents that Radar defaults to showing HTML content type only — which is relevant because the bot-vs-human ratio differs significantly depending on whether you look at all request types or HTML only.

Documentation checklist

image

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/radar/ @cdeath, @rubenalex, @cloudflare/radar, @cloudflare/product-owners

@ask-bonk ask-bonk Bot added the documentation Documentation edits label Jun 3, 2026
Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

Adds a clear glossary entry for Radar's bot vs. human traffic classification and the default HTML-only filter. One structural issue to fix before merge.

LOW: The new section is placed before the BGP entries. It should appear after and before to maintain alphabetical order.

Comment thread src/content/docs/radar/glossary.mdx Outdated
Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

Adds a clear glossary entry for Radar's bot vs. human traffic classification and the default HTML-only filter. One structural issue to fix before merge.

LOW: The new Bot vs. human traffic section is placed before the BGP entries. It should appear after ## BGP real-time routes and before ## Certificates to maintain alphabetical order.

Comment thread src/content/docs/radar/glossary.mdx Outdated
@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented Jun 3, 2026

Applied labels product:radar and documentation. Posted a review requesting changes.

Issue found:

  • LOW (src/content/docs/radar/glossary.mdx): The new Bot vs. human traffic section is out of alphabetical order. It is inserted before the BGP sections, but BGP precedes Bot alphabetically. It should be moved after ## BGP real-time routes and before ## Certificates.

The entry content itself is accurate and follows the style guide — links, formatting, and prose are all correct.

github run

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

@cdeath cdeath force-pushed the cdeath/radar-glossary-bot-vs-human branch from 18dda6c to aab129d Compare June 3, 2026 16:15
@cdeath cdeath force-pushed the cdeath/radar-glossary-bot-vs-human branch from aab129d to 908fbb9 Compare June 3, 2026 16:17
@cdeath cdeath marked this pull request as ready for review June 3, 2026 16:27
@cdeath cdeath requested review from a team and rubenalex as code owners June 3, 2026 16:27
@cloudflare-docs-bot
Copy link
Copy Markdown

cloudflare-docs-bot Bot commented Jun 3, 2026

Review

✅ No style-guide issues found in full PR diff.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.

@cdeath
Copy link
Copy Markdown
Collaborator Author

cdeath commented Jun 3, 2026

/full-review

@cdeath
Copy link
Copy Markdown
Collaborator Author

cdeath commented Jun 3, 2026

/bonk

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented Jun 3, 2026

Here's a summary of the thread on PR #31214:

What changed
cdeath added a "Bot vs. human traffic" glossary entry to the Radar glossary page. It explains how Radar classifies traffic (using bot scores) and notes that Radar defaults to showing HTML content type only — which matters because the bot-to-human ratio differs significantly across content types.

Review feedback
ask-bonk approved the content and style but requested one structural change: the new section was inserted before the ## BGP entries, violating alphabetical order. It should be moved after ## BGP real-time routes and before ## Certificates.

Author response
cdeath acknowledged the issue and pushed a fix ("fixed").

Current status
The alphabetical-order issue appears resolved. A cloudflare-docs-bot review failed transiently and is expected to retry automatically on the next push. andre-j3sus also reviewed but left no comment body.

github run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants