bugfix - fixed content safety client creation w/ APIM & managed identity

[Xeelee33]

A logic bug in config.py prevents the content_safety_client from being created unless a content safety API key is present in the settings because the if condition required both the endpoint and the key before continuing to client creation:

simplechat/application/single_app/config.py

Line 648 in cebd8f7

if safety_endpoint and safety_key:

This bugfix simply removes the "and safety_key" from the condition so the clients can be created properly whether the authentication uses API key, managed identity, or APIM.

Tested using the prompt from Microsoft's docs here: https://microsoft.github.io/TechExcel-Operationalize-LLMs-with-LLMOps-automation/docs/01_intro_to_llms/01_03.html#:~:text=and%20select%20Run-,Test,-.

Behavior before change:

Behavior after change:

[Copilot]

Pull request overview

This pull request fixes a bug in the content safety client initialization logic that prevented the client from being created when using managed identity or APIM authentication. The original code incorrectly required both the endpoint AND the API key to be present, blocking scenarios where authentication is handled via managed identity or APIM subscription keys.

Changes:

• Removed the and safety_key condition from the content safety client initialization check, allowing the client to be created with only the endpoint present
• This enables proper client creation for all three authentication methods: API key, managed identity, and APIM

[paullizer]

nice fix!