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

Threadfix #100

Closed
wants to merge 4 commits into from
Closed

Threadfix #100

wants to merge 4 commits into from

Conversation

elentar
Copy link

@elentar elentar commented Apr 10, 2017

Without this fix, the -t/--threads argument is always overwritten with either 12 or the number of cpus. This is at least annoyingly slow for larger host lists, and potentially dangerous when users expect to iterate over one host at at time e.g. for a cluster restart.

Update AUTHORS to use my personal email.

Pradeep Sanders and others added 4 commits August 31, 2016 09:34
…ional, and make tests pass with optional use_shell param

Fix expected size in test

Update manpage for no-shell option

exit -> true/false in tests

Add ability to set ssh options. Fixes worr#13

remove -Werror from default build options

Add -D_DEFAULT_SOURCE for modern platforms

Don't leak information about -r if not built with range

Document --ssh-opt

Add idle thread time. Possibly fixes worr#77

Figure out threads to run automatically

Add Pradeep to authors

Set use_shell to optional, for backward compatibility

Raise use_shell flag in command request if passed, now that it is optional, and make tests pass with optional use_shell param

Set use_shell to optional, for backward compatibility
@coveralls
Copy link

Coverage Status

Coverage remained the same at 77.355% when pulling eda6689 on elentar:threadfix into 3655f65 on worr:master.

@worr
Copy link
Owner

worr commented Apr 13, 2017

Applied manually in 689f5bd

@worr worr closed this Apr 13, 2017
@elentar elentar deleted the threadfix branch April 13, 2017 18:23
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.

None yet

3 participants