-
Notifications
You must be signed in to change notification settings - Fork 10.4k
Closed
Labels
api-approvedAPI was approved in API review, it can be implementedAPI was approved in API review, it can be implementedarea-networkingIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractionsIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Milestone
Description
The BCL is planning a replacement for this type: dotnet/runtime#79946
We should consume theirs when available and obsolete ours.
public class IPNetwork |
public IList<IPNetwork> KnownNetworks { get; } = new List<IPNetwork>() { new IPNetwork(IPAddress.Loopback, 8) }; |
Consider adding an implicit converter to minimize the breaking changes in the short term.
Proposed API
namespace Microsoft.AspNetCore.Builder;
public class ForwardedHeadersOptions
{
+ [Obsolete(...)]
public IList<Microsoft.AspNetCore.HttpOverrides.IPNetwork> KnownNetworks { get; }
+ public IList<System.Net.IPNetwork> KnownIPNetworks { get; }
}
aetos382, WeihanLi, silkfire, stylpe and pentp
Metadata
Metadata
Assignees
Labels
api-approvedAPI was approved in API review, it can be implementedAPI was approved in API review, it can be implementedarea-networkingIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractionsIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions