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

Diagnostics: Adds Client Configuration for Synchronization context cases #4338

Merged

Conversation

sourabh1007
Copy link
Contributor

Description

It was found that, Client Configuration was missing from request diagnostics if calls are made with synchronization context.

As part of this PR, fixing that.

Type of change

  • [] Bug fix (non-breaking change which fixes an issue)

@sourabh1007 sourabh1007 force-pushed the users/sourabhjain/incldudeclientconfig branch from dd2ca33 to f253aab Compare March 5, 2024 02:47
@ealsur ealsur changed the title Diagnostics: Adds Client Configuration for Sync Call also Diagnostics: Adds Client Configuration for Synchronization context cases Mar 5, 2024
@sourabh1007 sourabh1007 added the auto-merge Enables automation to merge PRs label Mar 5, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot merged commit d1a7d49 into master Mar 5, 2024
21 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot deleted the users/sourabhjain/incldudeclientconfig branch March 5, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Enables automation to merge PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants