Changes to modules/security/bot-analyzer.sh:
Problem:
- baseline_health_check() was re-checking HTTP/HTTPS status for all domains
- verify_domains_still_working() was re-testing domains again
- Wasteful duplicate checks when data already cached in reference database
Solution:
- baseline_health_check() now uses get_all_domain_statuses() from reference DB
- verify_domains_still_working() now uses get_domain_status() from reference DB
- Eliminated all curl HTTP status checks for local domains
- Significantly faster execution (no network requests needed)
Benefits:
- Instant baseline loading (uses pre-cached data from launcher startup)
- No redundant HTTP/HTTPS requests
- Consistent with toolkit architecture (centralized status collection)
- Same functionality, better performance
Technical Details:
- Uses get_all_domain_statuses() to load all domain status data
- Uses get_domain_status() to check individual domain status
- Returns same data format: domain|http_code|https_code|status_summary
- Added cache age warning in verify function (max 1 hour old)
- Maintains all existing baseline/verification logic
Note: Acronis scripts unchanged - they check external cloud URLs, not local domains
Performance Impact:
- Before: ~3-5 seconds per domain check (HTTP + HTTPS curl requests)
- After: Instant (reads from .sysref cache file)
- For 50 domains: ~5 minutes saved per execution
cschantz
0fa5676bac