fix: Document InnerHandler null requirement for DI (#1569)

bbrandt · glennawatson · web-flow · commit b59977fe40a2 · 2023-10-01T15:21:47.000+11:00
* Addressing #1403 Co-authored-by: Glenn <5834289+glennawatson@users.noreply.github.com>
diff --git a/README.md b/README.md
@@ -599,7 +599,9 @@ some interface `ITenantProvider` and has a data store `IAuthTokenStore` that can
     {
          this.tenantProvider = tenantProvider ?? throw new ArgumentNullException(nameof(tenantProvider));
          this.authTokenStore = authTokenStore ?? throw new ArgumentNullException(nameof(authTokenStore));
-         InnerHandler = new HttpClientHandler();
+         // InnerHandler must be left as null when using DI, but must be assigned a value when
+         // using RestService.For<IMyApi>
+         // InnerHandler = new HttpClientHandler();
     }
 
     protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)

Original file line number	Diff line number	Diff line change
@@ -599,7 +599,9 @@ some interface `ITenantProvider` and has a data store `IAuthTokenStore` that can
`599`	`599`	`{`
`600`	`600`	`this.tenantProvider = tenantProvider ?? throw new ArgumentNullException(nameof(tenantProvider));`
`601`	`601`	`this.authTokenStore = authTokenStore ?? throw new ArgumentNullException(nameof(authTokenStore));`
`602`		`- InnerHandler = new HttpClientHandler();`
	`602`	`+ // InnerHandler must be left as null when using DI, but must be assigned a value when`
	`603`	`+ // using RestService.For<IMyApi>`
	`604`	`+ // InnerHandler = new HttpClientHandler();`
`603`	`605`	`}`
`604`	`606`
`605`	`607`	`protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)`