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

Second ctrl+c should force quit the program #89

Closed
cyb3rjerry opened this issue Nov 15, 2022 · 4 comments
Closed

Second ctrl+c should force quit the program #89

cyb3rjerry opened this issue Nov 15, 2022 · 4 comments
Assignees
Labels
bug Something isn't working Go

Comments

@cyb3rjerry
Copy link
Contributor

Describe the bug
CTRL+C once initiates the "smooth" exit correctly. However, pressing it a second time should "force quit" as it currently simply hangs

To Reproduce
Steps to reproduce the behavior:

  1. Run any long scan
  2. Press CTRL+C twice
  3. See error

Expected behavior
Force quit on second CTRL+C

Screenshots
image

Desktop (please complete the following information):
Linux cyb3rjerry 5.19.0-23-generic 24-Ubuntu SMP PREEMPT_DYNAMIC Fri Oct 14 15:39:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

@cyb3rjerry
Copy link
Contributor Author

Crawling google and trying to leave for example causes this
image

@edoardottt
Copy link
Owner

Can you confirm this c547128 is fixing this issue? @cyb3rjerry

@edoardottt edoardottt added bug Something isn't working Go labels Nov 15, 2022
@cyb3rjerry
Copy link
Contributor Author

@edoardottt LGTM however we might also want to halt printing URLs to avoid confusion
Either that or add a sleep before printing the exit message to make sure it gets printed at the bottom
image

@edoardottt
Copy link
Owner

edoardottt commented Nov 15, 2022

Those are the responses to the requests that have been made before pressing CTRL+C and still didn't get a response. Then when they get it, they print them. What about adding os.Stdout = nil? I don't know what could be a working solution..

I can't even reproduce this bug

@edoardottt edoardottt self-assigned this Feb 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Go
Projects
None yet
Development

No branches or pull requests

2 participants