Feature #11666: IPAddr#private?

• Status: Closed
• Assignee: knu (Akinori MUSHA)
• Target version: -

[Glass_saga (Masaki Matsushita)]

I propose a new feature: IPAddr#private?.

Example:

IPAddr.new("192.168.1.1").private? #=> true.

[matz (Yukihiro Matsumoto)]

Accepted. But we have naming issue. private? or private_addr? or something else?
Any comments?

Matz.

[Glass_saga (Masaki Matsushita)]

I think private? is better than private_addr? because private_addr? seems to be redundant in following code:

IPAddr.new("192.168.1.1").private_addr?

[naruse (Yui NARUSE)]

• Status changed from Open to Assigned
• Assignee changed from Glass_saga (Masaki Matsushita) to knu (Akinori MUSHA)

The maintainer is knu.

• feature issues
  • conflict with private visibility
    • “private?” is different from visivility
  • Behavior when address family is IPv6
    • nil? false?
    • or treat link-local, site-local (deprecated), unique-local as true?
  • Naming differencies with Addrinfo
• The name “ipv4_private?” is clear that the method returns false on IPv6 addresses.

[herwin (Herwin W)]

IPv6 has a private range too: d00::/8 (RFC 4193)

Also, there are more IP ranges that have been reserved for special use (there's an overview on https://en.wikipedia.org/wiki/Reserved_IP_addresses). I think it would be better to make 1 method that can return a pre-defined symbol of that range if the IP address is in that range. Otherwise, there would be an explosion in the number of IP addresses.

[shyouhei (Shyouhei Urabe)]

As far as I understand RFC4193 never defines its range being "private". There also are other private-ish address ranges e.g. fd00::/8. And what about v4-mapped v6 addresses? IPv6's situation is complicated (at least to me). I'd suggest OP to leave IPv6 situations untouched.

[shyouhei (Shyouhei Urabe)]

• Has duplicate Feature #10912: Add method(s) to IPAddr for determining whether an address is link local added

[knu (Akinori MUSHA)]

• Status changed from Assigned to Closed

Applied in changeset trunk|r60270.

---

Import ipaddr 1.2.0

• Add IPAddr#prefix
• Add IPAddr#loopback?
• Add IPAddr#private? [Feature #11666]
• Add IPAddr#link_local? [Feature #10912]
• Reject invalid address mask [Bug #13399]
• Warn that IPAddr#ipv4_compat and #ipv4_compat? are deprecated [#Bug 13769]

[knu (Akinori MUSHA)]

Added, thanks for the suggestion.