Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: IPv6 subnet calculator #85

Closed
unquietwiki opened this issue Apr 23, 2018 · 9 comments
Closed

Feature request: IPv6 subnet calculator #85

unquietwiki opened this issue Apr 23, 2018 · 9 comments

Comments

@unquietwiki
Copy link

There are several implementations to reference; and its possible code from the first example may be license-compatible.

@BornToBeRoot
Copy link
Owner

Or maybe this...

https://github.com/lduchosal/ipnetwork

Available via Nuget. BSD 2

@BornToBeRoot BornToBeRoot added this to the 1.6.3.0 milestone May 11, 2018
@BornToBeRoot
Copy link
Owner

Subnet calculator for ipv6 added (no ipv6 splitter)

Commit: 0f46f58

@unquietwiki
Copy link
Author

@BornToBeRoot I was assuming you wanted me to build & test that on my end. My build environment doesn't seem to like building it. I tried to improvise, but results were mixed. Here's from my abortive attempt to make it work: using VS Code, .NET 4.7.2, and .NET Core 2.1 SDKs.

@BornToBeRoot
Copy link
Owner

Hi,

which VS and Windows version do you use?

There is a Pre Build script which generates the AxMSTSCLib.dll and MSTSCLib.dll, but you can also copy these files from one of the other releases into the programfolder.
https://github.com/BornToBeRoot/NETworkManager/blob/master/Scripts/PreBuildEventCommandLine.ps1

I plan to add the subnetting and supernet feature this weekend: https://github.com/lduchosal/ipnetwork
This will be the last feature for the version 1.6.3.0, then I will publish it

The subnet calculator for IPv6 is working (this libary is tested and used by many others), I attached a build of the current master.

NETworkManager_v1.6.3.0_BETA.zip

@unquietwiki
Copy link
Author

@BornToBeRoot build spec already provided; funny enough, VS Code was saying I didn't have the stuff for 4.6 SDK installed; which I thought I did. (shrugs) You may want to try re-targeting to 4.7.2 anyway, given the recent update + support for .NET Standard 2.0 content.

The calculator works, though one thought: if I put in 2001:DB8:: , I should get a /32 result; not a half-hearted /64. From my experience with IPv6, a lot of calculation is going to take place above /64; so use-cases of /32, /48, and /56 may be more common than past /64. You may even want to do some soliciting of potential users for more feedback.

@BornToBeRoot
Copy link
Owner

BornToBeRoot commented Jun 5, 2018

There is an open issue in the IPNetwork libary (7 days ago), let's wait what for a response from the maintainer.

lduchosal/ipnetwork#29

I think it would make sense if the user always have to enter the cidr 2001:db8:/xx. I have to change the validator that no IPv6 without CIDR can be entered...

Btw. The Subnetting feature looks good, but the application freezes on large subnets (like 0/ to /128 😄 )

image

I will make this async and add a warning/note.

@unquietwiki
Copy link
Author

@BornToBeRoot Awesomesauce. And yeah, a warning would be good. :p

@BornToBeRoot
Copy link
Owner

Subnet Calculator / Subnetting / Supernetting with IPv6 support added!

Version 1.6.3.0 released: https://github.com/BornToBeRoot/NETworkManager/releases/latest

@unquietwiki
Copy link
Author

@BornToBeRoot I saw you posted an updated choco package too. May want to look into using https://chocolatey.org/packages/dotnetfx as a dependency, instead of the version-specific one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants