Consider adding IPNetwork.Parse

[svick]

Is your feature request related to a problem? Please describe.

I've been reading a blog post by Jon Skeet and noticed this code in it:

options.KnownNetworks.Add(new IPNetwork(IPAddress.Parse("130.211.0.0"), 22));

I think that this code is quite verbose and it would be easier to understand if it would be possible to use IP subnet syntax here.

Describe the solution you'd like

A new method:

public class IPNetwork
{
    public static IPNetwork Parse(string subnetString);
}

It could be used like this:

options.KnownNetworks.Add(IPNetwork.Parse("130.211.0.0/22")));

If this method was added, it would probably also make sense to add a Try version of it:

public static bool TryParse(string subnetString, out IPNetwork network);

What do you think?

[blowdart]

This would probably fit better in .NET Core, along with the rest of the IPAddress classes. Might I suggest asking there?

[svick]

@blowdart The IPNetwork class is already part of ASP.NET Core, I'm just suggesting adding few methods to it.

Or are you saying the whole class should be moved to .Net Core?

[blowdart]

In my opinion, yes. Of course other may disagree :) Feels weird it's in ASP.NET when it could be used elsewhere

[khellang]

There's already a corefx issue that's related to this.