💚 Update Application Insights to use Connection String instead of Instrumentation Key

SSWConsulting / SSW.People

This is the site generator for SSW.People. ▶️ Video: http://r.ssw.com/SSW-people

https://ssw.com.au/people

MIT License

12 stars 10 forks source link

💚 Update Application Insights to use Connection String instead of Instrumentation Key #648

Closed jernejk closed 3 months ago

jernejk commented 3 months ago

cc: @wicksipedia @adamcogan

Microsoft changed the recommended way of configuring Application Insights from using Instrumentation Keys to Connection Strings. As per my conversation with @wicksipedia, we should update this.

### Tasks
- [x] Update the code that loads the connection string
- [x] Update environment variables from `APPINSIGHTS_INSTRUMENTATIONKEY` to `APPINSIGHTS_CONNECTIONSTRING`
- [x] Update GitHub environment variables

jernejk commented 3 months ago

Done but needs #651 to be deployable.