v2 - OpenApiModelFactory.GetStreamAsync should accept a cancellation token

[baywet]

OpenAPI.NET/src/Microsoft.OpenApi/Reader/OpenApiModelFactory.cs

Line 300 in cb5bb27

private static async Task<Stream> GetStreamAsync(string url)

To pass it to GetStreamAsync. (conditional compilation on NET5 or higher).

All callers to this method should pass the value, and accept one.

This is important as if the action is being cancelled on a backend service, this will forward the cancellation to the HTTP request being made, saving important COGS in the process.

[MaggieKimani1]

The core library only targets netstandard 2.0

[baywet]

My suggestion is to add additional targets and conditional compilation to better support high-throughput scenarios.

We should probably have a separate conversation about the netstandard2.0 target which gives us netfx + a bunch of end of life net core versions (when compared with lets say net6).

[RachitMalik12]

#1975