-
Notifications
You must be signed in to change notification settings - Fork 561
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Newsletters Module #1072
Newsletters Module #1072
Conversation
I will be resolving the merge conflicts (via command line) as well as modifying the test_module_newsletters. I'd like to make two checks, one for WITH newsletters and one for WITHOUT newsletters. I currently have one check that fails if either one doesn't pass. |
… for event.data['host']
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #1072 +/- ##
=====================================
+ Coverage 92% 92% +1%
=====================================
Files 304 306 +2
Lines 19306 19357 +51
=====================================
+ Hits 17760 17807 +47
- Misses 1546 1550 +4 ☔ View full report in Codecov by Sentry. |
bbot/modules/newsletters.py
Outdated
class newsletters(BaseModule): | ||
watched_events = ["HTTP_RESPONSE"] | ||
produced_events = ["FINDING"] | ||
flags = ["passive", "safe"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"passive" here should be "active" since it relies on web requests.
Yippie! |
Description:
Ticket/Issue
To Review:
bbot -m httpx newsletters -t blacklanternsecurity.com
bbot -m httpx newsletters -t futureparty.com
Notes:
bbot -m httpx newsletters -t <insert target here>