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

Add Bandit Security Linting to Github Actions #17

Merged
merged 7 commits into from
May 26, 2024
Merged

Conversation

maxachis
Copy link

@maxachis maxachis commented May 26, 2024

Fixes

Description

  • A new GitHub Actions workflow 'bandit.yaml' has been added to run Bandit, a Python security linter, on pull requests.
  • This will increase the security standards of the codebase by ensuring that potential security vulnerabilities are addressed promptly.
  • The Bandit results will be uploaded as an artifact for further analysis

Testing

  • Can be evaluated based on Github Action output

Performance

  • Takes less than a minute to run using Github Actions

Docs

  • Not applicable.

A new GitHub Actions workflow 'bandit.yaml' has been added to run Bandit, a Python security linter, on push and pull requests. This will increase the security standards of the codebase by ensuring that potential security vulnerabilities are addressed promptly. The Bandit results will be uploaded as an artifact for further analysis.
maxachis added 6 commits May 26, 2024 08:03
Removed the explicit exit command from the Bandit execution command in the GitHub Actions workflow.
The command to print out Bandit's output directly to the console in the GitHub Actions workflow was removed. Now, the results are only being uploaded as an artifact for review.
Changed the triggering event of Bandit Security Linting workflow from happening on both push and pull_request events to only on pull_request events.
Updated the Bandit Security Linting code to also include 'resources' and 'app.py', expanding the range of files which are scanned for potential security vulnerabilities. Previously, linting was only applied to 'middleware'.
@maxachis maxachis merged commit 07318bb into dev May 26, 2024
9 of 10 checks passed
@maxachis maxachis deleted the mc_issue_292_bandit branch May 26, 2024 22:36
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.

1 participant