refactor: fix critical NameErrors, clean up unused imports/variables,…#530
Open
dkoryto wants to merge 2 commits into
Open
refactor: fix critical NameErrors, clean up unused imports/variables,…#530dkoryto wants to merge 2 commits into
dkoryto wants to merge 2 commits into
Conversation
… and modernize config - Fix missing 'import subprocess' in wifite/attack/wpa.py (NameError) - Fix missing 'log_error' import in wifite/util/interface_assignment.py (NameError) - Fix missing 'InterfaceManager' import in wifite/util/system_check.py (NameError) - Fix color variables in f-strings in wifite/util/wpa3_tools.py (NameError) - Restore 'import subprocess' in wifite/util/cleanup.py (needed by tests) - Remove dead code/duplicate entry-point from setup.py - Remove deprecated flake8-ignore from setup.cfg - Add psutil to dependencies (pyproject.toml + requirements.txt) - Remove sys.path.insert hacks from 16 test files - Mass cleanup of unused imports (F401), unused variables (F841), and empty f-strings (F541) across wifite/ and tests/ Test results: 617 passed, 7 failed (pre-existing/environment issues)
- kimocoder#520: Verify OpenCL devices are actually available before using wpapsk-opencl format in john.py (not just compiled in) - kimocoder#477: Uncomment --rds=3 and --enable_status=15 flags in hcxdumptool.py passive mode to prevent aggressive default behavior in hcxdumptool v7 - kimocoder#522: In infinite mode, return empty target list instead of raising Exception when no targets are found, allowing scan cycle to continue
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
… and modernize config
Test results: 617 passed, 7 failed (pre-existing/environment issues)