Added exception handler for read timeouts

[PeterMosmans]

Added a basic exception handler for timeouts from the requests module.

[SamJoan]

Hi @PeterMosmans

thanks for your contribution, I will look into it before droopescan's next release. Any ideas on why tests are failing for this branch?

If you could add tests for the new functionality that'd be awesome too

[PeterMosmans]

Hi @droope , thanks. I haven't looked at tests (yet), and I'm unsure as to why the tests are failing. Does your code expects a different error (assertion) when no connection can be made ?

[SamJoan]

Hi

I think the tests are failing because functionality is failing. If I scan an example site with your branch of the plugin:

[root@debian-dev droopescanpeter (timeouthandler)]# ./droopescan scan -u www.example.org -n 100 -e p
[+] Accepted redirect to https://www.example.org/
[+] Site identified as drupal.
[+] No plugins found.

With the master branch:

[root@debian-dev droopescan (master)]# ./droopescan scan -u www.example.org -n 100 -e p
[+] Accepted redirect to https://www.example.org/
[+] Site identified as drupal.
[+] Plugins found:
    views https://www.example.org/sites/all/modules/views/
        https://www.example.org/sites/all/modules/views/README.txt
        https://www.example.org/sites/all/modules/views/LICENSE.txt
    ctools https://www.example.org/sites/all/modules/ctools/
        https://www.example.org/sites/all/modules/ctools/CHANGELOG.txt
        https://www.example.org/sites/all/modules/ctools/API.txt
    [SNIP..]

I think giving a warning instead of throwing an exception would be in line with how droopescan operates (e.g. with 500 errors), so if you add a test and fix the error I'm happy to merge it. The test is likely to be similar to this one: https://github.com/droope/droopescan/blob/master/dscan/tests/base_http_tests.py#L440

[SamJoan]

Hi

I've implemented the change you requested (a warning instead of a immediate failure on read timeouts.)

The code changes will be present on droopescan's next release.