Skip to content

Conversation

@bcdady
Copy link
Contributor

@bcdady bcdady commented Jun 26, 2025

This PR adds automated static-analysis security scanning with Bandit to both local workflows and CI, and resolves the first high-severity issue it uncovered.

  • Adds dedicated “Security scan with Bandit” step that fails the build on ≥ low-level findings.

Security Fix (B701)

  • Enabled autoescape=True in tfsumpy/plan/reporter.py Jinja2 Environment, eliminating Bandit B701 (potential XSS).

fix minor linting issue in test_cli.py

bcdady added 3 commits June 26, 2025 01:08
…dit B701)

Bandit flagged B701 because Jinja2’s Environment defaulted to autoescape=False.
Setting autoescape=True prevents potential XSS issues when rendering templates
@bcdady bcdady marked this pull request as ready for review June 26, 2025 07:15
@rafaelherik rafaelherik merged commit 6587ac4 into rafaelherik:main Jun 26, 2025
4 checks passed
@rafaelherik rafaelherik assigned rafaelherik and bcdady and unassigned rafaelherik Jun 26, 2025
@rafaelherik rafaelherik added enhancement New feature or request security It's a security related issue/PR. labels Jun 26, 2025
@bcdady bcdady deleted the ci-bandit branch June 27, 2025 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request security It's a security related issue/PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants