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

Added certificates parsing, Python Flask web viewer and update requir… #57

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

x4v1l0k
Copy link

@x4v1l0k x4v1l0k commented Aug 25, 2024

Added certificates parsing.
Python Flask web viewer.
Update requirements.txt.
Update files from main branch to dev branch.

@r4ulcl
Copy link
Owner

r4ulcl commented Aug 25, 2024

Hi, I really like the idea and how it's implemented so far. However, could you move all the Flask related code into a separate file within the utils folder to keep everything cleaner?

Additionally, it would be great if the server execution could be controlled with a boolean flag, allowing the IP and port to be passed as parameters, defaulting to 127.0.0.1 instead of 0.0.0.0.

Also, please review the Codacy messages in the pull request to check for any vulnerabilities or errors.

utils/wifi_db_aircrack.py Dismissed Show dismissed Hide dismissed
utils/wifi_db_aircrack.py Dismissed Show dismissed Hide dismissed
utils/wifi_db_aircrack.py Dismissed Show dismissed Hide dismissed
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