Azure-Samples / contoso-real-estate

Intelligent enterprise-grade reference architecture for JavaScript, featuring OpenAI integration, Azure Developer CLI template and Playwright tests.
MIT License
889 stars 528 forks source link

Secrets leak fix (do not merge) #400

Closed dfberry closed 11 months ago

dfberry commented 12 months ago

Fixes #386: by moving off v3 to v4 for api - used git mv instead of mv

Fixes #350 1) remove 3 secrets from Azure Function environment so they don't show up in log, 2) remove 3 seconds from infra output, 3) Add 3 secrets with correct names to Key Vault from infra 4) read Key Vault in ./secrets for db restore, wrapper bash script to read kv and restore db 5) don't need to do anything for API code because it was already reading Key Vault as part of config

With this PR, both blog and portal work again when deployed.