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.

Sign up for GitHub

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

Jump to bottom

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

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

ivarne opened this issue Oct 13, 2023 · 0 comments · Fixed by #379
Assignees
@ivarne @tjololo
Labels
kind/chore
Milestone
8.0.0

Comments

@ivarne
Copy link
Member

ivarne commented Oct 13, 2023 •
edited
Loading

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
@ivarne ivarne mentioned this issue Dec 13, 2023
Merged
5 tasks
@tjololo tjololo linked a pull request Dec 19, 2023 that will close this issue
Support using connectionString for ApplicationInsights #379
Merged
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
Assignees

@ivarne ivarne

@tjololo tjololo

Labels
kind/chore
Projects
Team Apps
Archived in project
Milestone
8.0.0
Development

Successfully merging a pull request may close this issue.

Support using connectionString for ApplicationInsights Altinn/app-lib-dotnet
3 participants
@ivarne @tjololo @RonnyB71