Feature #11210
closedIPAddr has no public method to get the current subnet mask
Description
Both to_s and to_string omit the subnet mask of an IP address. The only way to query it via public methods is to manually parse the output of inspect.
Updated by bjmllr (Ben Miller) over 9 years ago
It would be useful to expose @mask_addr. I have done this at times by reopening IPAddr and adding an attr_reader. Then the string form of the mask can be obtained with
IPAddr.new(addr.mask_addr, addr.family).to_s
#mask_addr is also useful when doing subnetting operations which are beyond the scope of IPAddr.
Updated by Eregon (Benoit Daloze) over 8 years ago
- Assignee set to knu (Akinori MUSHA)
Assigning this to the official maintainer of ipaddr.
Or should it be assigned to matz?
Could this be decided at the next Developer meeting or sooner?
See https://github.com/ruby/ruby/pull/1269 for the patch.
Updated by knu (Akinori MUSHA) over 8 years ago
- Status changed from Open to Assigned
Sorry for being inactive.
I've made some comments in the above PR.
I can't recall the exact reason why there's no access to the netmask provided, but one main design concept of IPAddr by the original author (Ume-san) was to allow manipulating IP addresses independent of what family each instance is in, so I'd welcome the PR in that it introduces an address family independent class for returning values.
Except for a few things I mentioned in the PR it looks good to merge, so please toss your feedback to it if you have any opinions. Thank you.
Updated by herwin (Herwin W) about 8 years ago
Any progress? I believe all the comments have been addressed.
Updated by usa (Usaku NAKAMURA) over 7 years ago
- Related to Feature #11527: IPAddr#mask_addr isn't a method added
Updated by Eregon (Benoit Daloze) about 7 years ago
@knu (Akinori MUSHA): Could you review the PR and decide whether to merge it?
Updated by herwin (Herwin W) over 6 years ago
https://github.com/ruby/ipaddr/commit/293ad5ac6ed704f91d418ecaf24ae58156a587e7 fixes this in the ipaddr gem. Good enough for me, I guess this issue can be closed.
Updated by knu (Akinori MUSHA) almost 6 years ago
- Status changed from Assigned to Closed