inetaf / netaddr

Network address types

add ParseIPRange, IPRange.String?

bradfitz opened this issue 4 years ago · comments

Brad Fitzpatrick commented 4 years ago

I've implemented my own ParseIPRange a few times now. Add to this package?

And corresponding IPRange.String() in form from.String() + "-" + to.String()?

Objections?

/cc @danderson @mdlayher @josharian

Josh Bleecher Snyder commented 4 years ago

SGTM

Brad Fitzpatrick commented 4 years ago

e.g. here's one of the existing IPRange parsers: https://github.com/tailscale/tailscale/blob/70f14af21eaa130f0f677d30310951715c9498eb/wgengine/filter/tailcfg.go#L97

There's another in the coordinator server.