final code push #22
Open
final code push #22
Conversation
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.
CyberScan Python 3 Migration - PR Description
Summary
This PR migrates CyberScan from Python 2.7 to Python 3, resolving critical compatibility issues that prevented the script from running on modern systems. The original code was failing with
ImportError: No module named pcapyand numerous Python 2 syntax errors. The solution involved running thelib2to3conversion tool to update deprecated syntax (exception handling, print statements, long integers, dictionary methods), fixing removed modules (UserDict,new), and replacing the bundled outdatedscapy_locallibrary with the modernscapypackage from PyPI. Additionally, PowerShell execution policies were configured to allow virtual environment activation on Windows.The migration also addresses Windows-specific requirements for network scanning operations. Changes include proper error handling for missing Npcap installations, updated documentation with administrator privilege requirements, and a clean dependency management system using
requirements.txt. Testing confirms all core functionality works correctly: help menu displays properly, ICMP pings succeed (tested with 8.8.8.8), and port scanning operates as expected when run with administrator privileges. The PR includes comprehensive README updates documenting Python 3.8+ requirements, step-by-step Windows setup instructions, Npcap installation guidance, and solutions for common errors like execution policy blocks and raw socket permission issues.Files Changed:
CyberScan.py(updated imports),requirements.txt(added),.gitignore(added),README.md(updated),scapy_local(2to3 conversion applied)Testing: Verified on Windows 10 with Python 3.x | All scanning modes functional | Documentation complete