Skip to content

fix(ci): replace global B113 bandit skip with per-file annotations (#2272)#2289

Merged
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-2272
Mar 24, 2026
Merged

fix(ci): replace global B113 bandit skip with per-file annotations (#2272)#2289
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-2272

Conversation

@mrveiss
Copy link
Copy Markdown
Owner

@mrveiss mrveiss commented Mar 24, 2026

Summary

  • Removes B113 (request_without_timeout) from global .bandit skips
  • Adds per-line # nosec B113 to test files that use requests.*:
    • infrastructure.integration_test.py (9 calls)
    • phase8_control_panel_test.py (5 calls)
    • voice_integration_test.py (7 calls — bandit false positive, all have timeout=)
  • Future production code using requests.* without timeout will now be caught by CI
  • Verified: bandit -c .bandit -r autobot-backend/ autobot-slm-backend/ autobot-shared/ -t B113 returns 0 issues

Closes #2272

@github-actions
Copy link
Copy Markdown

✅ SSOT Configuration Compliance: Passing

🎉 No hardcoded values detected that have SSOT config equivalents!

@mrveiss mrveiss merged commit 543e67e into Dev_new_gui Mar 24, 2026
4 of 5 checks passed
@mrveiss mrveiss deleted the fix/issue-2272 branch March 26, 2026 16:56
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