Skip to content

Commit d9b1fd9

Browse files
authored
Don't include the Request Headers in the UnexpectedHttpStatusCodeException message anymore (#369)
Adjust UnexpectedHttpStatusCodeException message building: Don't use HttpRequestMessage.ToString anymore, it includes the header values e.g. "Authorization". Now append all the request properties ourself, except for Headers.
1 parent 24112f1 commit d9b1fd9

1 file changed

Lines changed: 13 additions & 2 deletions

File tree

Source/EasyNetQ.Management.Client/UnexpectedHttpStatusCodeException.cs

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,20 @@ public static async Task<UnexpectedHttpStatusCodeException> FromHttpResponseMess
6363
sb.Append("<null>");
6464
}
6565
sb.Append(" from request: ");
66-
if (response.RequestMessage != null)
66+
var request = response.RequestMessage;
67+
if (request != null)
6768
{
68-
sb.Append(response.RequestMessage);
69+
sb.Append("Method: ");
70+
sb.Append(request.Method.ToString());
71+
72+
sb.Append(", RequestUri: '");
73+
sb.Append(request.RequestUri == null ? "<null>" : request.RequestUri.ToString());
74+
75+
sb.Append("', Version: ");
76+
sb.Append(request.Version.ToString());
77+
78+
sb.Append(", Content: ");
79+
sb.Append(request.Content == null ? "<null>" : request.Content.GetType().ToString());
6980
}
7081
else
7182
{

0 commit comments

Comments
 (0)