Skip to content

fix: guard against chaotic chaos responses#3078

Open
TrebledJ wants to merge 1 commit intoblacklanternsecurity:devfrom
TrebledJ:patch-7
Open

fix: guard against chaotic chaos responses#3078
TrebledJ wants to merge 1 commit intoblacklanternsecurity:devfrom
TrebledJ:patch-7

Conversation

@TrebledJ
Copy link
Copy Markdown
Contributor

@TrebledJ TrebledJ commented May 4, 2026

In extremely rare cases, Chaos API returns a gigaton of subdomains. If that happens, we simply treat the domain as a wildcard. I've hardcoded the limit to 10,000 subdomains, which I think is a bit generous, but oh well.

resolves #2879

@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 91%. Comparing base (5be4993) to head (1a08d2f).
⚠️ Report is 13 commits behind head on dev.

Files with missing lines Patch % Lines
bbot/modules/chaos.py 50% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##             dev   #3078   +/-   ##
=====================================
- Coverage     91%     91%   -0%     
=====================================
  Files        437     437           
  Lines      37509   37511    +2     
=====================================
- Hits       33925   33924    -1     
- Misses      3584    3587    +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

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