Skip to content

fix(security): harden outbound client configuration in update checker and hub sync - #1495

Merged
Wikid82 merged 9 commits into
developmentfrom
fix/localhost-allowance-followups
Oct 6, 2026
Merged

Wikid82 merged 9 commits into
developmentfrom
fix/localhost-allowance-followups

Conversation

@Wikid82

@Wikid82 Wikid82 commented Oct 5, 2026

Copy link
Copy Markdown
Owner

Summary

Follow-up to #1492. Removes or contains the remaining production uses of a testing-only allowance in outbound clients:

  • Update checker: the allowance is removed; the default client is always strict. Tests inject their own client.
  • Hub sync: the allowance moves behind an unexported test-only seam.
  • Uptime checks / LAPI registration: comments only, clarifying the intentional design (monitors may target local services).
  • Removes unused LAPI health helpers and their tests (no non-test callers).

Behaviour note

A CrowdSec hub mirror configured via HUB_BASE_URL / HUB_MIRROR_BASE_URL can no longer point at a loopback address. Documented in docs/troubleshooting/crowdsec.md.

Testing

  • Regression tests for the strict default clients and the test-only seam
  • go test -race ./internal/services/... ./internal/crowdsec/..., go test ./internal/api/..., make lint-fast, pre-commit hooks pass
  • Backend coverage 92.4%; patch coverage 100%
  • QA report: docs/reports/qa_report.md

Built with help from Claude Code.

@github-advanced-security

Copy link
Copy Markdown
Contributor

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@codecov

codecov Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

✅ Supply Chain Verification Results

✅ PASSED

📦 SBOM Summary

  • Components: 1870

🔍 Vulnerability Scan

Severity Count
🔴 Critical 0
🟠 High 0
🟡 Medium 0
🟢 Low 0
Total 0

📎 Artifacts

  • SBOM (CycloneDX JSON) and Grype results available in workflow artifacts

Generated by Supply Chain Verification workflow • View Details

@Wikid82
Wikid82 merged commit 5a63907 into development Oct 6, 2026
47 checks passed
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.

2 participants