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

Save checked-for-existance commands inside a list to avoid rechecking them more than once #141

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

codeswhite
Copy link
Contributor

@codeswhite codeswhite commented Sep 10, 2018

This was done as a result of my terminal being spammed with:
Executing: which hashcat
And various other tool existence-checks when using -vv option

All the changes works great on ArchLinux - which may be added as a supported distribution.
This time I've done unit tests which passed successfully (except for pyrit & cowpatty handshake validation).

Note: I encourage to check this PR for python2 compatibility, as some of the changes might not work well with py2 (like removing the trailing backslash in attack/wep.py:225)

WhiteOnBlackCode added 24 commits September 3, 2018 00:28
* Do not show handshake files that are in cracked.txt with a key (match on filename).
* Don't ask user for a crack-tool when attacking PMKIDs only
* Few minor cleanups
…hashcat)

* Added a safe-check to make sure we are indeed using hashcat for the PMKID hashes
* Changed the ugly split() to basename()
…hashcat)

* Added a safe-check to make sure we are indeed using hashcat for the PMKID hashes
* Changed the ugly split() to basename()
# Conflicts:
#	wifite/util/crack.py
# Conflicts:
#	wifite/args.py
#	wifite/config.py
#	wifite/util/crack.py
# Conflicts:
#	wifite/config.py
# Conflicts:
#	wifite/config.py
…ver and over again (-vvv not spammed anymore)
actually skip pmkid crack if we dont have hashcat installed
Make sure all changes are compatible with python2
@RaduNico
Copy link

Hello!

It might be useful to clean up this PR a bit (removing the merge commits perhaps?) it is a bit difficult to navigate them as they are now. It might be useful to just cherry-pick your commits onto a clean branch, maybe merge some of the minor changes together. Cheers!

cbrunnkvist pushed a commit to cbrunnkvist/wifite2 that referenced this pull request Jun 4, 2023
…s-67.6.1

build(deps): bump setuptools from 67.6.0 to 67.6.1
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

2 participants