Skip to content

Commit 44bf8cd

Browse files
committed
[dotnet] Don't include http headers in internal logs
1 parent 7d1d0e0 commit 44bf8cd

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

dotnet/src/webdriver/Remote/HttpCommandExecutor.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
using System;
2222
using System.Collections.Generic;
2323
using System.Globalization;
24+
using System.Linq;
2425
using System.Net;
2526
using System.Net.Http;
2627
using System.Net.Http.Headers;
@@ -417,7 +418,7 @@ protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage
417418
var responseTask = base.SendAsync(request, cancellationToken);
418419

419420
StringBuilder requestLogMessageBuilder = new();
420-
requestLogMessageBuilder.AppendFormat(">> {0}", request);
421+
requestLogMessageBuilder.AppendFormat(">> {0} RequestUri: {1}, Content: {2}, Headers: {3}", request.Method, request.RequestUri, request.Content, request.Headers?.Count());
421422

422423
if (request.Content != null)
423424
{
@@ -430,7 +431,7 @@ protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage
430431
var response = await responseTask.ConfigureAwait(false);
431432

432433
StringBuilder responseLogMessageBuilder = new();
433-
responseLogMessageBuilder.AppendFormat("<< {0}", response);
434+
responseLogMessageBuilder.AppendFormat("<< StatusCodde: {0}, ReasonPhrase: {1}, Content: {2}, Headers: {3}", (int)response.StatusCode, response.ReasonPhrase, response.Content, response.Headers?.Count());
434435

435436
if (!response.IsSuccessStatusCode && response.Content != null)
436437
{

0 commit comments

Comments
 (0)