Skip to content

Commit 74fda5c

Browse files
committed
Some more minor refactoring
1 parent bb5f8cd commit 74fda5c

File tree

6 files changed

+20
-12
lines changed

6 files changed

+20
-12
lines changed

Patros.AuthenticatedHttpClient.AuthorizationHeader/AuthorizationHeaderAuthenticatedHttpClientOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@
22
{
33
public class AuthorizationHeaderAuthenticatedHttpClientOptions
44
{
5-
public string Value;
5+
public string Value { get; set; }
66
}
77
}

Patros.AuthenticatedHttpClient.CustomHeader/CustomHeaderAuthenticatedHttpMessageHandler.cs

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,16 @@ namespace Patros.AuthenticatedHttpClient
66
{
77
public class CustomHeaderAuthenticatedHttpMessageHandler : DelegatingHandler
88
{
9-
private CustomHeaderAuthenticatedHttpClientOptions _options;
9+
private readonly CustomHeaderAuthenticatedHttpClientOptions _options;
1010

11-
public CustomHeaderAuthenticatedHttpMessageHandler(CustomHeaderAuthenticatedHttpClientOptions options, HttpMessageHandler innerHandler = null)
11+
public CustomHeaderAuthenticatedHttpMessageHandler(CustomHeaderAuthenticatedHttpClientOptions options)
1212
{
13-
InnerHandler = innerHandler ?? new HttpClientHandler();
13+
_options = options;
14+
}
15+
16+
public CustomHeaderAuthenticatedHttpMessageHandler(CustomHeaderAuthenticatedHttpClientOptions options, HttpMessageHandler innerHandler)
17+
{
18+
InnerHandler = innerHandler;
1419

1520
_options = options;
1621
}

Patros.AuthenticatedHttpClient.CustomHeader/MultipleCustomHeaderAuthenticatedHttpClientOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
namespace Patros.AuthenticatedHttpClient
44
{
55
public class MultipleCustomHeaderAuthenticatedHttpClientOptions {
6-
public Dictionary<string, string> Headers;
6+
public Dictionary<string, string> Headers { get; set; }
77
}
88
}

Patros.AuthenticatedHttpClient.QueryStringParameter/MultipleQueryStringParameterAuthenticatedHttpClientOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
namespace Patros.AuthenticatedHttpClient
44
{
55
public class MultipleQueryStringParameterAuthenticatedHttpClientOptions {
6-
public Dictionary<string, string> Parameters;
6+
public Dictionary<string, string> Parameters { get; set; }
77
}
88
}
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
namespace Patros.AuthenticatedHttpClient
22
{
33
public class QueryStringParameterAuthenticatedHttpClientOptions {
4-
public string Name;
5-
public string Value;
4+
public string Name { get; set; }
5+
public string Value { get; set; }
66
}
77
}

Patros.AuthenticatedHttpClient.QueryStringParameter/QueryStringParameterAuthenticatedHttpMessageHandler.cs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,18 @@ namespace Patros.AuthenticatedHttpClient
88
{
99
public class QueryStringParameterAuthenticatedHttpMessageHandler : DelegatingHandler
1010
{
11-
private QueryStringParameterAuthenticatedHttpClientOptions _options;
11+
private readonly QueryStringParameterAuthenticatedHttpClientOptions _options;
1212

13-
public QueryStringParameterAuthenticatedHttpMessageHandler(QueryStringParameterAuthenticatedHttpClientOptions options, HttpMessageHandler innerHandler = null)
13+
public QueryStringParameterAuthenticatedHttpMessageHandler(QueryStringParameterAuthenticatedHttpClientOptions options)
1414
{
15-
InnerHandler = innerHandler ?? new HttpClientHandler();
16-
1715
_options = options;
1816
}
1917

18+
public QueryStringParameterAuthenticatedHttpMessageHandler(QueryStringParameterAuthenticatedHttpClientOptions options, HttpMessageHandler innerHandler) : this(options)
19+
{
20+
InnerHandler = innerHandler;
21+
}
22+
2023
protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
2124
{
2225
var escapedValue = Uri.EscapeDataString(_options.Value);

0 commit comments

Comments
 (0)