Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Application Insights will require connection string (instead of instrumentation key) in 2025 #325

Closed
2 tasks
ivarne opened this issue Oct 13, 2023 · 0 comments · Fixed by #379
Closed
2 tasks
Assignees
Milestone

Comments

@ivarne
Copy link
Member

ivarne commented Oct 13, 2023

Description

See: microsoft/ApplicationInsights-dotnet#2560

We should ensure that version 8 includes the required changes to set connection string in Altinn.App.Api.Extentions.ServiceCollectionExtentions.AddApplicationInsights, and consider updating the app environment as well

Additional Information

Consider documenting how to set the app insights log level lower than Error as described in #223

Tasks

  • Update app template to use ApplicationInsights__ConnectionString if available.
  • Update app environment to set ApplicationInsights__ConnectionString in addition to ApplicationInsights__InstrumentationKey.

Acceptance Criterias

New apps on version 8 is ready for the deprecation of InstrumentationKey based ingestion.

@ivarne ivarne added status/draft Status: When you create an issue before you have enough info to properly describe the issue. kind/chore labels Oct 13, 2023
@ivarne ivarne added this to the 8.0.0 milestone Oct 13, 2023
@tjololo tjololo linked a pull request Dec 19, 2023 that will close this issue
5 tasks
@RonnyB71 RonnyB71 removed the status/draft Status: When you create an issue before you have enough info to properly describe the issue. label Jan 2, 2024
@tjololo tjololo closed this as completed Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants