Skip to content

senzing-factory/build-resources#185 add spellcheck#186

Merged
docktermj merged 1 commit into
mainfrom
sf-br-185-skern
Jun 11, 2025
Merged

senzing-factory/build-resources#185 add spellcheck#186
docktermj merged 1 commit into
mainfrom
sf-br-185-skern

Conversation

@kernelsam
Copy link
Copy Markdown
Contributor

Pull request questions

Which issue does this address

Issue number: senzing-factory/build-resources#185

@kernelsam kernelsam requested review from a team as code owners June 10, 2025 21:15
@github-actions
Copy link
Copy Markdown

🛡️ Bandit Scan Results Summary

We found 2 High, 2 Medium, and 1 Low severity issues.

Detailed Findings


Severity Issue File Line Confidence More Info Test ID
🟡 LOW Consider possible security implications associated with the subprocess module. sz_tools/_tool_helpers.py 14 HIGH More Info B404
⚪ MEDIUM Possible SQL injection vector through string-based query construction. sz_tools/_sz_database.py 354 LOW More Info B608
⚪ MEDIUM Probable insecure usage of temp file/directory. sz_tools/_tool_helpers.py 764 MEDIUM More Info B108
🔴 HIGH Starting a process with a shell, possible injection detected, security issue. sz_tools/_tool_helpers.py 688 HIGH More Info B605
🔴 HIGH subprocess call with shell=True identified, security issue. sz_tools/_tool_helpers.py 671 HIGH More Info B602

✨ About this Report

This report was generated by the official Bandit GitHub Action to ensure our codebase stays secure.

📕 What is Bandit?

Bandit is a tool designed to find common security issues in Python code. To learn more about how Bandit helps to keep Python code safe, visit the Bandit documentation.

👥 Community Support

Got questions or need help with Bandit Action?

  • Join our community on the Discord server.
  • Share tips, get advice, and collaborate on security best practices.

@docktermj docktermj merged commit 1f20864 into main Jun 11, 2025
27 of 83 checks passed
@docktermj docktermj deleted the sf-br-185-skern branch June 11, 2025 13:14
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.

4 participants