Skip to content

[minor] Add verified Common Crawl bypass - #108

Merged
joecorall merged 2 commits into
mainfrom
common-crawl-ip-check
Aug 29, 2026
Merged

[minor] Add verified Common Crawl bypass#108
joecorall merged 2 commits into
mainfrom
common-crawl-ip-check

Conversation

@joecorall

Copy link
Copy Markdown
Member

Allow a config option (enabled by default) to allow Common CRAWL bot through.

The config follows the docs at https://commoncrawl.org/ccbot by grabbing the list of IPs at https://index.commoncrawl.org/ccbot.json and doing a reverse DNS lookup on the IPv4 ranges before adding the client IPs to the list of good bots.

@joecorall
joecorall force-pushed the common-crawl-ip-check branch from 5feec72 to cfcbf68 Compare August 29, 2026 10:15
@joecorall
joecorall enabled auto-merge (squash) August 29, 2026 10:17
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.91589% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/helper/commoncrawl.go 84.90% 8 Missing ⚠️
main.go 88.88% 6 Missing ⚠️
Files with missing lines Coverage Δ
main.go 85.38% <88.88%> (+4.05%) ⬆️
internal/helper/commoncrawl.go 84.90% <84.90%> (ø)

... and 8 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@joecorall
joecorall merged commit be16c2f into main Aug 29, 2026
7 checks passed
@joecorall
joecorall deleted the common-crawl-ip-check branch August 29, 2026 10:18
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