Azure-Samples / langfuse-on-azure

An easy way to deploy the Langfuse observability platform to Azure Container Apps with Entra authentication.
MIT License
39 stars 3 forks source link

Make sure all secrets are secure #9

Closed pamelafox closed 8 months ago

pamelafox commented 8 months ago

Purpose

This PR removes security alerts caused by two issues- authClientSecret not being stored in secrets, and secrets not being marked as a secure input in the Bicep.

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test