-
-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Problem
autobot-infrastructure/shared/scripts/setup/analytics/setup_seq_analytics.py contains ~46 print() calls throughout the file. The project standard requires logging.getLogger(__name__) for all output — print() is blocked by pre-commit in production code.
This file was partially fixed in PR #2531 (hardcoded password removed), but the print() calls were out of scope for that security fix.
Discovered During
Code review of PR #2531 (working on #2348 — clear-text logging)
Impact
Severity: low — infrastructure setup script, not hot-path production code. But violates project coding standard and bypasses log rotation/aggregation.
Fix
Replace all print() calls with logger.info() / logger.error() using the standard pattern:
import logging
logger = logging.getLogger(__name__)Reactions are currently unavailable