Skip to content

Commit

Permalink
adding headers and query parms for logging (#22537)
Browse files Browse the repository at this point in the history
  • Loading branch information
Mohamed Shaban authored Jul 12, 2021
1 parent 3321330 commit cd5d1e7
Showing 1 changed file with 27 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ public class DocumentTranslationClientOptions : ClientOptions
public DocumentTranslationClientOptions(ServiceVersion version = LatestVersion)
{
Version = version;
AddLoggedHeadersAndQueryParameters();
}

internal string GetVersionString()
Expand All @@ -52,5 +53,31 @@ public enum ServiceVersion
/// </summary>
V1_0 = 1
}

/// <summary>
/// Add headers and query parameters that are considered safe for logging or including in
/// error messages by default.
/// </summary>
private void AddLoggedHeadersAndQueryParameters()
{
Diagnostics.LoggedHeaderNames.Add("Operation-Location");
Diagnostics.LoggedHeaderNames.Add("Content-Encoding");
Diagnostics.LoggedHeaderNames.Add("Vary");
Diagnostics.LoggedHeaderNames.Add("apim-request-id");
Diagnostics.LoggedHeaderNames.Add("X-RequestId");
Diagnostics.LoggedHeaderNames.Add("Set-Cookie");
Diagnostics.LoggedHeaderNames.Add("X-Powered-By");
Diagnostics.LoggedHeaderNames.Add("Strict-Transport-Security");
Diagnostics.LoggedHeaderNames.Add("x-content-type-options");

Diagnostics.LoggedQueryParameters.Add("$top");
Diagnostics.LoggedQueryParameters.Add("$skip");
Diagnostics.LoggedQueryParameters.Add("$maxpagesize");
Diagnostics.LoggedQueryParameters.Add("ids");
Diagnostics.LoggedQueryParameters.Add("statuses");
Diagnostics.LoggedQueryParameters.Add("createdDateTimeUtcStart");
Diagnostics.LoggedQueryParameters.Add("createdDateTimeUtcEnd");
Diagnostics.LoggedQueryParameters.Add("$orderBy");
}
}
}

0 comments on commit cd5d1e7

Please sign in to comment.