-
Notifications
You must be signed in to change notification settings - Fork 129
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
No certificate transparency error handling leads to crash #78
Labels
bug
Something isn't working
Comments
Hey! Thanks for reporting this potential bug. I will take a look at it. |
@atenreiro simple fix is to add the below at app.py:439
|
Awesome, done! Next time feel free to create a Pull, and I'll merge it 🙂 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is a very useful utility, thank you. I've noticed that when using the --ct switch, the script fails on found domains that don't have TLS configured on port 443 (or at all):
[*] HTTPSConnectionPool(host='REDACTED.com', port=443): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f9879699bd0>: Failed to establish a new connection: [Errno 110] Connection timed out'))
Traceback (most recent call last):
File "./opensquat.py", line 75, in
args.ct
File "/tools/opensquat/opensquat/app.py", line 633, in main
return self.worker()
File "/tools/opensquat/opensquat/app.py", line 368, in worker
domains
File "/tools/opensquat/opensquat/app.py", line 439, in _process_doppelgagner_only
if self.response_contains_keyword(keyword):
File "/tools/opensquat/opensquat/app.py", line 423, in response_contains_keyword
return keyword in self.response.text
AttributeError: 'Domain' object has no attribute 'response'
The text was updated successfully, but these errors were encountered: