Skip to content

Commit e6e7a0c

Browse files
committed
Add null checks
1 parent f21e953 commit e6e7a0c

File tree

1 file changed

+13
-4
lines changed

1 file changed

+13
-4
lines changed

src/RestSharp.Extensions.DependencyInjection/ServiceCollectionExtensions.cs

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,9 @@ public void AddRestClient(
1717
ConfigureRestClient? configureRestClient = null,
1818
ConfigureSerialization? configureSerialization = null
1919
) {
20+
Ensure.NotEmptyString(name, nameof(name));
21+
Ensure.NotNull(services, nameof(services));
22+
2023
var options = new RestClientOptions();
2124
var configure = configureRestClient ?? (_ => { });
2225
configure(options);
@@ -69,20 +72,26 @@ public void AddRestClient(
6972
/// </summary>
7073
/// <param name="options">Custom options for the RestClient.</param>
7174
[PublicAPI]
72-
public void AddRestClient(RestClientOptions options) => services.AddRestClient(Constants.DefaultRestClient, o => o.CopyFrom(options));
73-
75+
public void AddRestClient(RestClientOptions options) {
76+
Ensure.NotNull(options, nameof(options));
77+
services.AddRestClient(Constants.DefaultRestClient, o => o.CopyFrom(options));
78+
}
79+
7480
/// <summary>
7581
/// Adds a named RestClient to the service collection with base URL.
7682
/// </summary>
7783
/// <param name="name">Client name.</param>
7884
/// <param name="baseUrl">The base URL for the RestClient.</param>
7985
public void AddRestClient(string name, Uri baseUrl) => services.AddRestClient(name, o => o.BaseUrl = baseUrl);
80-
86+
8187
/// <summary>
8288
/// Adds a named RestClient to the service collection with custom options.
8389
/// </summary>
8490
/// <param name="name">Client name.</param>
8591
/// <param name="options">Custom options for the RestClient.</param>
86-
public void AddRestClient(string name, RestClientOptions options) => services.AddRestClient(name, o => o.CopyFrom(options));
92+
public void AddRestClient(string name, RestClientOptions options) {
93+
Ensure.NotNull(options, nameof(options));
94+
services.AddRestClient(name, o => o.CopyFrom(options));
95+
}
8796
}
8897
}

0 commit comments

Comments
 (0)