Skip to content

Commit ab01890

Browse files
Add request tracing for push refresh usage (#626)
* in progress * complete adding ispushrefreshused logic * rename pushrefresh tag string * remove empty line
1 parent 7dc547c commit ab01890

File tree

4 files changed

+16
-0
lines changed

4 files changed

+16
-0
lines changed

src/Microsoft.Extensions.Configuration.AzureAppConfiguration/AzureAppConfigurationProvider.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -548,6 +548,11 @@ public void ProcessPushNotification(PushNotification pushNotification, TimeSpan?
548548

549549
if (_configClientManager.UpdateSyncToken(pushNotification.ResourceUri, pushNotification.SyncToken))
550550
{
551+
if (_requestTracingEnabled && _requestTracingOptions != null)
552+
{
553+
_requestTracingOptions.IsPushRefreshUsed = true;
554+
}
555+
551556
SetDirty(maxDelay);
552557
}
553558
else

src/Microsoft.Extensions.Configuration.AzureAppConfiguration/Constants/RequestTracingConstants.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ internal class RequestTracingConstants
3333
public const string LoadBalancingEnabledTag = "LB";
3434
public const string SignalRUsedTag = "SignalR";
3535
public const string FailoverRequestTag = "Failover";
36+
public const string PushRefreshTag = "PushRefresh";
3637

3738
public const string FeatureFlagFilterTypeKey = "Filter";
3839
public const string CustomFilter = "CSTM";

src/Microsoft.Extensions.Configuration.AzureAppConfiguration/RequestTracingOptions.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,11 @@ internal class RequestTracingOptions
6363
/// </summary>
6464
public bool IsFailoverRequest { get; set; } = false;
6565

66+
/// <summary>
67+
/// Flag to indicate whether push refresh is used.
68+
/// </summary>
69+
public bool IsPushRefreshUsed { get; set; } = false;
70+
6671
/// <summary>
6772
/// Checks whether any tracing feature is used.
6873
/// </summary>

src/Microsoft.Extensions.Configuration.AzureAppConfiguration/TracingUtils.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -201,6 +201,11 @@ private static string CreateCorrelationContextHeader(RequestType requestType, Re
201201
correlationContextTags.Add(RequestTracingConstants.FailoverRequestTag);
202202
}
203203

204+
if (requestTracingOptions.IsPushRefreshUsed)
205+
{
206+
correlationContextTags.Add(RequestTracingConstants.PushRefreshTag);
207+
}
208+
204209
var sb = new StringBuilder();
205210

206211
foreach (KeyValuePair<string, string> kvp in correlationContextKeyValues)

0 commit comments

Comments
 (0)