Alan-Jowett / CoPilot-For-Consensus / 20687668634

78%

push

github

fix: Add AZURE_CLIENT_ID environment variable to all microservices for managed identity authentication (#714)

* fix: Add AZURE_CLIENT_ID environment variable to all microservices for managed identity authentication

Adds AZURE_CLIENT_ID to parsing, chunking, embedding, orchestrator, and summarization services. Reporting was already fixed. Ingestion already had it. This resolves the managed identity authentication failures preventing services from connecting to Cosmos DB, Service Bus, and Key Vault.

References #713

* fix: Add COSMOS_DB configuration to auth service role store

Auth service was defaulting to MongoDB for role store instead of Cosmos. Add:
- AUTH_ROLE_STORE_TYPE: cosmos
- COSMOS_DB_ENDPOINT: Cosmos DB endpoint

This enables the auth service to use the same Cosmos DB backend as other services for persisting user role assignments.

* fix: Filter out MongoDB parameters when creating Cosmos DB document store

Services were passing MongoDB-specific parameters (host, port, username, password) to create_document_store even when configured to use Cosmos DB. This caused those parameters to be passed to the Azure Cosmos SDK, which doesn't understand them, resulting in:

  TypeError: Session.request() got an unexpected keyword argument 'host'

Now filter out MongoDB-specific parameters when creating a Cosmos store to prevent them from being passed to the SDK.

---------

Co-authored-by: GitHub Copilot <copilot@github.com>

3 of 3 new or added lines in 1 file covered. (100.0%)

6014 of 7878 relevant lines covered (76.34%)

0.85 hits per line