Skip to content

Conversation

@mo-radwan1
Copy link
Collaborator

Summary

  • Added /api/v1/platform/health to the list of paths that skip OAuth authentication
  • This allows load balancers (ALB, NGINX) and monitoring systems (Kubernetes probes, external monitors) to check Platform Service health without requiring authentication

Changes

  • Modified src/solace_agent_mesh/shared/auth/middleware.py to include /api/v1/platform/health in the skip_paths list

Behavior

  • Platform Service health endpoint (/api/v1/platform/health): ✅ No authentication required
  • Gateway health endpoints (/api/v1/platform/gateways/health, /api/v1/platform/gateways/{id}/health): 🔒 Authentication still required
  • Gateway discovery endpoint (/api/v1/platform/gateways): 🔒 Authentication still required

Rationale

The Platform Service health endpoint is designed for infrastructure monitoring and should be accessible without authentication to ensure:

  • Load balancers can perform health checks
  • Kubernetes liveness/readiness probes work correctly
  • External monitoring systems can track service availability

Gateway monitoring endpoints remain protected as they expose operational data about the gateway fleet.

Test Plan

  • Verify /api/v1/platform/health is accessible without authentication
  • Verify gateway health endpoints still require authentication
  • Test with load balancer health checks
  • Test Kubernetes probe configuration if deployed

🤖 Generated with Claude Code

…thentication

- Add /api/v1/platform/health to skip_paths in OAuth middleware
- Allows load balancers and monitoring systems to check Platform Service health without authentication
- Gateway health endpoints (/api/v1/platform/gateways/health) still require authentication

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@sonarqube-solacecloud
Copy link

@mo-radwan1 mo-radwan1 merged commit cb333b4 into main Jan 7, 2026
20 checks passed
@mo-radwan1 mo-radwan1 deleted the mradwan/DATAGO-121143-exclude-platform-health-from-auth branch January 7, 2026 20:07
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.

3 participants