Skip to content

Conversation

@joecorall
Copy link
Member

@joecorall joecorall commented Dec 12, 2025

Adding reverse DNS for the google bot also includes allowing customers from google cloud unchallenged. So instead lets just grab google's IPs and treat that as a good bot.

@github-actions
Copy link

github-actions bot commented Dec 12, 2025

Running State Persistence Stress Tests

Testing Small, Medium, Large, and XLarge scales...

Stress Test Summary

Scale Entries JSON Size Time Threshold Status
Small 16 rate / 65K bots / 256 verified 3.87 MB 141ms <500ms
Medium 256 rate / 262K bots / 65K verified 19.31 MB 765ms <1s
Large 1K rate / 1M bots / 262K verified 77.61 MB 3903ms <3s
XLarge 4K rate / 4.2M bots / 1M verified 312.68 MB 18261ms <10s

ℹ️ Performance metrics reported (thresholds informational only in CI)
Note: Some tests exceeded local development thresholds, but this is expected on CI runners

@codecov-commenter
Copy link

codecov-commenter commented Dec 12, 2025

Codecov Report

❌ Patch coverage is 45.23810% with 46 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
main.go 10.25% 33 Missing and 2 partials ⚠️
internal/helper/google.go 75.55% 6 Missing and 5 partials ⚠️
Files with missing lines Coverage Δ
internal/helper/google.go 75.55% <75.55%> (ø)
main.go 73.51% <10.25%> (-4.38%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@joecorall joecorall changed the title Add google bot IP "firewall" [minor] Add google bot IP passthru [minor] Dec 12, 2025
@joecorall joecorall force-pushed the googlebot branch 2 times, most recently from 6220f46 to ef17ce7 Compare December 12, 2025 20:53
@joecorall joecorall merged commit 7077b67 into main Dec 12, 2025
11 checks passed
@joecorall joecorall deleted the googlebot branch December 12, 2025 21:36
joecorall added a commit that referenced this pull request Dec 13, 2025
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.

3 participants