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 scanning to pipeline [RHELDST-12100] #22

Merged
merged 1 commit into from
Jul 3, 2022
Merged

Add Bandit scanning to pipeline [RHELDST-12100] #22

merged 1 commit into from
Jul 3, 2022

Conversation

sahensley
Copy link
Contributor

To enable SAST scanning on this repository, Bandit has been added into
the tox.ini and .travis.yml files. Two tests are executed in the
pipeline: py3-bandit-exitzero and py3-bandit.

The first scan lists all findings of low severity or higher and always
passes due to the "exit-zero" option. This will allow tracking of low
severity findings without stopping code from being merged in.

The second scan lists all findings of medium severity or higher and will
fail the pipeline if any issues have been introduced.

To track the overall progress of Bandit enablement on Pub open-source
libraries, see Jira [RHELDST-9236].

To enable SAST scanning on this repository, Bandit has been added into
the tox.ini and .travis.yml files.  Two tests are executed in the
pipeline: py3-bandit-exitzero and py3-bandit.

The first scan lists all findings of low severity or higher and always
passes due to the "exit-zero" option.  This will allow tracking of low
severity findings without stopping code from being merged in.

The second scan lists all findings of medium severity or higher and will
fail the pipeline if any issues have been introduced.

To track the overall progress of Bandit enablement on Pub open-source
libraries, see Jira [RHELDST-9236].
@sahensley sahensley marked this pull request as ready for review July 1, 2022 22:08
@sahensley
Copy link
Contributor Author

Travis is configured to use Python 3.8 for the Bandit steps due to Bandit having a hard requirement of Python version >= 3.7.

@rohanpm @rbikar @rajulkumar - Please review. Thank you!

@rohanpm rohanpm self-requested a review July 3, 2022 23:03
@rohanpm rohanpm merged commit cceaae9 into release-engineering:master Jul 3, 2022
@sahensley sahensley deleted the add-bandit branch July 5, 2022 17:02
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.

None yet

2 participants