Skip to content

Merge PR #5710: Expand OTel exception handling for Azure Functions compatibility#5720

Merged
neha-bhargava merged 3 commits intomainfrom
copilot/merge-fork-to-new-branch
Feb 6, 2026
Merged

Merge PR #5710: Expand OTel exception handling for Azure Functions compatibility#5720
neha-bhargava merged 3 commits intomainfrom
copilot/merge-fork-to-new-branch

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 5, 2026

Changes proposed in this request

Merges changes from PR #5710 (fork: fowl2:patch-1) into AzureAD repo branch to enable CI test execution.

Core change: Expands exception handling in AbstractPlatformProxy.InternalGetOtelInstrumentation() to catch TypeLoadException and TypeInitializationException alongside existing FileNotFoundException.

catch (Exception ex) 
when (ex is FileNotFoundException
         or TypeLoadException
         or TypeInitializationException)

Context: In-process Azure Functions with older System.Diagnostics.DiagnosticSource versions throw TypeLoadException when OTel initialization fails. Previously only FileNotFoundException was caught, causing unhandled exceptions.

Testing

Changes originate from external contributor PR. CI workflows now executing on this branch to validate compatibility across target environments.

Performance impact

None. Exception handling path only executes on initialization failure.

Documentation

  • All relevant documentation is updated.
Original prompt

Create a new branch and merge the changes from tis PR to that branch. Then the tests will run on that branch as the tests do not run on forks, so we merge that branch. Help me do that. We will simply merge this fork to a new branch and merge the new branch to main


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Merge changes from fork into new branch Merge PR #5710: Expand OTel exception handling for Azure Functions compatibility Feb 5, 2026
Copilot AI requested a review from neha-bhargava February 5, 2026 21:43
@neha-bhargava neha-bhargava marked this pull request as ready for review February 6, 2026 17:37
@neha-bhargava neha-bhargava requested a review from a team as a code owner February 6, 2026 17:37
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.

5 participants