DRAFT: Adds Transport Layer Header Improvements #3078
Conversation
asketagarwal
requested review from
kirankumarkolli,
j82w,
ealsur,
FabianMeiswinkel,
kirillg and
sboshra
as code owners
asketagarwal
changed the title
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Please follow the required format: "[Internal] Category: (Adds|Fixes|Refactors|Removes) Description"
Internal should be used for PRs that have no customer impact. This flag is used to help generate the changelog to know which PRs should be included. Examples:
Diagnostics: Adds GetElapsedClientLatency to CosmosDiagnostics
PartitionKey: Fixes null reference when using default(PartitionKey)
[v4] Client Encryption: Refactors code to external project
[Internal] Query: Adds code generator for CosmosNumbers for easy additions in the future.
| TransportSerialization.FillTokenFromHeader(request, HttpConstants.HttpHeaders.IsClientEncrypted, rntbdRequest.isClientEncrypted, rntbdRequest); | ||
| TransportSerialization.FillTokenFromHeader(request, WFConstants.BackendHeaders.CorrelatedActivityId, rntbdRequest.correlatedActivityId, rntbdRequest); | ||
| */ | ||
| internal interface IRequestHeaders |
| @@ -15,7 +15,7 @@ namespace Microsoft.Azure.Cosmos | |||
| using Microsoft.Azure.Documents; | |||
| using Microsoft.Azure.Documents.Collections; | |||
|
|
|||
| internal class StoreRequestNameValueCollection : CosmosMessageHeadersInternal, INameValueCollection | |||
| internal class StoreRequestNameValueCollection : CosmosMessageHeadersInternal, INameValueCollection, IRequestHeaders | |||
There was a problem hiding this comment.
This is a generated file. This needs to be changed in https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/Microsoft.Azure.Cosmos/src/Headers/StoreRequestNameValueCollection.tt
|
Moved: #3177 |
Draft PR for master repo that changes the way SDK lookups header from the INameValueCollection