Skip to content
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

Ping reply as condition to continue the test #15

Closed
mpwill opened this issue Nov 17, 2016 · 2 comments
Closed

Ping reply as condition to continue the test #15

mpwill opened this issue Nov 17, 2016 · 2 comments

Comments

@mpwill
Copy link

mpwill commented Nov 17, 2016

Hi! First of all, congratulations for the great App.

I would like to report a behavior I believe will help many users. In the HTTP_url.PM, the code dictates the app should terminate in case of a ping failure:

if(!$ping){
die "[-] Web server didn't respond !\n";

I was testing a windows box where firewall would not allow ping, and modifying the code allowed me to proceed with the test.

While by design, modifying the message to mention 'ping failure' as the cause would narrow down the issue for users.

Continue with your great projects! :)

@mpwill mpwill changed the title Ping reply as condition to continue the attack Ping reply as condition to continue the test Nov 17, 2016
@wireghoul
Copy link
Owner

wireghoul commented Nov 18, 2016

By ping do you mean ICMP ping like the command line ping utility? It's not entirely clear to me what problem you're describing...

@wireghoul
Copy link
Owner

While your bug report still seems to confuse whatever you experienced with some other network protocol the -C command line switch probably solves your problem.

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

No branches or pull requests

2 participants