Skip to content

Fix first-request good bot DNS verification#90

Merged
joecorall merged 2 commits into
mainfrom
fix/81-goodbots-dns
Jun 21, 2026
Merged

Fix first-request good bot DNS verification#90
joecorall merged 2 commits into
mainfrom
fix/81-goodbots-dns

Conversation

@joecorall

Copy link
Copy Markdown
Member

Closes #81

@github-actions

Copy link
Copy Markdown

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 / 256 verified 16.08 KB 2ms <500ms
Medium 256 rate / 65K verified 3.90 MB 99ms <1s
Large 1K rate / 262K verified 15.59 MB 538ms <3s
XLarge 4K rate / 1M verified 62.74 MB 2524ms <10s

ℹ️ Performance metrics reported (thresholds informational only in CI)

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.95652% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/helper/ip.go 84.21% 1 Missing and 2 partials ⚠️
Files with missing lines Coverage Δ
main.go 84.69% <100.00%> (+0.06%) ⬆️
internal/helper/ip.go 83.78% <84.21%> (-9.55%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@joecorall joecorall merged commit 4da7966 into main Jun 21, 2026
12 checks passed
@joecorall joecorall deleted the fix/81-goodbots-dns branch June 21, 2026 21:51
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.

goodBots DNS lookup is async, causing bots to be challenged on first request

2 participants