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

Fix timeout -k implementation error #725

Merged
merged 7 commits into from
Jun 13, 2023
Merged

Fix timeout -k implementation error #725

merged 7 commits into from
Jun 13, 2023

Conversation

nicksvv
Copy link
Contributor

@nicksvv nicksvv commented Jun 13, 2023

Hey @six2dez, that an error in timeout -k implentation, so right now Amass not working in some functions.

image
The timeout -k option takes two arguments:

  1. The first argument (DURATION_KILL) is the duration after which the timeout command should send a KILL signal to the process if it's still running after the initial timeout period has elapsed. This duration should be specified in the same format as the DURATION argument (e.g., '15s' for 15 seconds, '1m' for 1 minute, '2h' for 2 hours, '3d' for 3 days, etc.).
  2. The second argument is the initial timeout duration (DURATION) after which a TERM signal is sent to the process. The DURATION argument should be followed by the command to run and limit the execution time of.

So in the command timeout -k 1m ${AMASS_INTEL_TIMEOUT}m amass intel -d ${domain} -whois a TERM signal will be sent to the process after 15 minutes of execution. If the process continues to run 1 minute beyond this, a KILL signal will be dispatched to forcefully terminate it.

@six2dez six2dez changed the base branch from main to dev June 13, 2023 21:54
@six2dez six2dez merged commit d6fffc3 into six2dez:dev Jun 13, 2023
@six2dez
Copy link
Owner

six2dez commented Jun 13, 2023

Very well spotted! I don't know what I was thinking when I code that. Thanks a lot for contributing! Btw, I changed the branch to dev in order to follow the development process. Thanks again!

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

Successfully merging this pull request may close these issues.

2 participants