Skip to content

security: harden subprocess usage in loose_scripts#396

Open
yehorcallmedai-maker wants to merge 1 commit into
makehumancommunity:masterfrom
yehorcallmedai-maker:repomend/fix-bandit.B404-31a881
Open

security: harden subprocess usage in loose_scripts#396
yehorcallmedai-maker wants to merge 1 commit into
makehumancommunity:masterfrom
yehorcallmedai-maker:repomend/fix-bandit.B404-31a881

Conversation

@yehorcallmedai-maker

Copy link
Copy Markdown
Contributor

Bandit findings in src/loose_scripts/ (developer utility scripts):

  • B404import subprocess flagged; import removed where unused
  • B607 — partial executable path "grep" replaced with full path via shutil.which("grep")
  • B603 — explicit shell=False added to subprocess.run calls

No production code affected — loose_scripts are developer tooling only.

Detected and patched by RepoMend.

@joepal1976

Copy link
Copy Markdown
Contributor

Thanks. Same comment as for the other PR though: I only see one of the changes.

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