Skip to content

Commit

Permalink
Updating ClientTelemetry to NOT use NameValueCollectionWrapperFactory. (
Browse files Browse the repository at this point in the history
#3473)

NameValueCollectionWrapperFactory is dead code and getting removed.

ClientTelemetry is using NameValueCollectionWrapperFactory for auth header related generation. Its usage moved to StoreResponseNameValueCollection.
  • Loading branch information
kirankumarkolli committed Sep 27, 2022
1 parent caa10f7 commit 2a34beb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Microsoft.Azure.Cosmos/src/Telemetry/ClientTelemetry.cs
Original file line number Diff line number Diff line change
Expand Up @@ -316,7 +316,7 @@ private async Task SendAsync()

async ValueTask<HttpRequestMessage> CreateRequestMessage()
{
INameValueCollection headersCollection = new NameValueCollectionWrapperFactory().CreateNewNameValueCollection();
INameValueCollection headersCollection = new StoreResponseNameValueCollection();
await this.tokenProvider.AddAuthorizationHeaderAsync(
headersCollection,
endpointUrl,
Expand Down

0 comments on commit 2a34beb

Please sign in to comment.