Alan-Jowett / CoPilot-For-Consensus / 20388034807

78%

Build Type

push

github

Committed by web-flow

Commit Message

Integrate JWT authentication and RBAC across all microservices with optional disable (#451)

* Initial plan

* Add JWT middleware to all microservices with RBAC

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add auth integration tests and documentation

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix docker-compose syntax errors

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add authentication section comment to reporting requirements

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Address code review feedback: fix docs, test fixtures, and unused variables

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add jwt_auth_enabled config option to conditionally enable/disable auth

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix ImportError: make copilot_auth import conditional

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add httpx dependency for orchestrator test_auth_integration tests

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix PyJWT 2.x compatibility in auth integration tests

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* fix: implement mock auth middleware in test fixtures for all services

- Ingestion, Orchestrator, and Reporting services had auth integration test failures
- Tests were expecting 401/403 status codes but receiving 200 due to missing router registration
- Created fresh FastAPI apps in test fixtures with:
  - Essential endpoints (health, stats, /api routes)
  - Mock JWT authentication middleware for testing
  - Token validation (expiration, audience, roles)
  - Proper error responses (401, 403)
- Updated role requirements per service:
  - Ingestion: admin role
  - Orchestrator: orchestrator role
  - Reporting: reader role
- All 19 auth i... (continued)

Run Details

21 of 50 new or added lines in 2 files covered. (42.0%)

10 existing lines in 2 files now uncovered.

5153 of 6443 relevant lines covered (79.98%)

0.89 hits per line