Skip to content
2 changes: 1 addition & 1 deletion .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: Validate Deployment v3

on:
workflow_run:
workflows: ["Build Docker and Optional Push"]
workflows: ["Build Docker and Optional Push v3"]
types:
- completed
branches:
Expand Down
4 changes: 1 addition & 3 deletions src/backend/common/config/app_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -112,9 +112,7 @@ def get_azure_credential(self, client_id=None):
Credential object: Either DefaultAzureCredential or ManagedIdentityCredential.
"""
if self.APP_ENV == "dev":
return (
DefaultAzureCredential()
) # CodeQL [SM05139] Okay use of DefaultAzureCredential as it is only used in development
return DefaultAzureCredential() # CodeQL [SM05139]: DefaultAzureCredential is safe here
else:
return ManagedIdentityCredential(client_id=client_id)

Expand Down
2 changes: 1 addition & 1 deletion src/backend/v3/magentic_agents/reasoning_search.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ async def initialize(self, kernel: Kernel) -> bool:
return False

try:
credential = SyncDefaultAzureCredential()
# credential = SyncDefaultAzureCredential()
Comment thread
AjitPadhi-Microsoft marked this conversation as resolved.
Outdated

self.search_client = SearchClient(endpoint=self.search_config.endpoint,
credential=AzureKeyCredential(self.search_config.api_key),
Expand Down
3 changes: 2 additions & 1 deletion src/backend/v3/orchestration/orchestration_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,8 @@ async def init_orchestration(cls, agents: List, user_id: str = None)-> MagenticO
temperature=0.1
)

credential = SyncDefaultAzureCredential()
credential = config.get_azure_credential(client_id=config.AZURE_CLIENT_ID)
# credential = SyncDefaultAzureCredential()

def get_token():
token = credential.get_token("https://cognitiveservices.azure.com/.default")
Expand Down
Loading