Skip to content

fix(appinsights): set browser app id for correlation - #1266

Merged
BenjaminMichaelis merged 1 commit into
mainfrom
fix/appinsights-browser-correlation
Aug 9, 2026
Merged

fix(appinsights): set browser app id for correlation#1266
BenjaminMichaelis merged 1 commit into
mainfrom
fix/appinsights-browser-correlation

Conversation

@BenjaminMichaelis

Copy link
Copy Markdown
Member

Summary\n- expose the Application Insights app id to the browser\n- configure the browser SDK with the app id for client/server correlation\n- keep correlation headers limited to the web and Try origins\n\n## Testing\n- dotnet build EssentialCSharp.Web.slnx -c Release /p:AccessToNugetFeed=false

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the web app’s Application Insights browser instrumentation to support improved client/server correlation by exposing the AI App ID to the browser and configuring the JS SDK to use it, while keeping correlation headers restricted to the site and Try origins.

Changes:

  • Expose APPLICATIONINSIGHTS_APP_ID to the browser via _Layout.cshtml.
  • Configure the Application Insights browser SDK with appId and clarify the intent of the correlation header domain allow-list.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
EssentialCSharp.Web/wwwroot/js/appinsights-manager.js Reads the exposed App ID and passes it into the browser SDK config to support correlation; documents the restricted correlation header domain behavior.
EssentialCSharp.Web/Views/Shared/_Layout.cshtml Adds a serialized window.APPLICATIONINSIGHTS_APP_ID value so the browser telemetry manager can configure correlation.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@BenjaminMichaelis
BenjaminMichaelis merged commit 5e2092b into main Aug 9, 2026
8 checks passed
@BenjaminMichaelis
BenjaminMichaelis deleted the fix/appinsights-browser-correlation branch August 9, 2026 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants