8000 Add PyZMQ python package + some improvement to advance setup script by mr-mosi · Pull Request #130 · google/tsunami-security-scanner · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add PyZMQ python package + some improvement to advance setup script #130

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

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

Conversation

mr-mosi
Copy link
@mr-mosi mr-mosi commented Feb 28, 2025

@tooryx @maoning

during the setup, I noticed some lack of documentation for quick_start_advanced.sh, please review my commit, also this PR is mainly related to this Plugin

Also, requests_mock was not available as a requirement.

Why do I want to indicate the Python version? because if you have python3.12 in your OS, then quick_start_advanced.sh won't work correctly.

PR: google/tsunami-security-scanner-plugins#608

use python3.11 because python3.12 is not compatible
update .gitignore
@giacomo-doyensec
Copy link

Hello @mr-mosi, thank you for your contribution.
While I agree that Python 3.11 is required to run the script, hardcoding the version like this could interfere with users relying on tools like Python version managers.
Could you please remove the changes related to the advanced setup script?

@giacomo-doyensec
Copy link

Hello @mr-mosi, any update on this?

@mr-mosi
Copy link
Author
mr-mosi commented May 19, 2025

@giacomo-doyensec I reverted the changes for quick_start_advanced.sh file.

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
0