[wp-trac] [WordPress Trac] #34538: Improvement of the IPv4 address format check

WordPress Trac noreply at wordpress.org
Mon Nov 2 01:16:56 UTC 2015


#34538: Improvement of the IPv4 address format check
----------------------------------------+------------------------------
 Reporter:  ka2                         |       Owner:
     Type:  enhancement                 |      Status:  new
 Priority:  normal                      |   Milestone:  Awaiting Review
Component:  HTTP API                    |     Version:  trunk
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |     Focuses:
----------------------------------------+------------------------------

Comment (by ka2):

 Thank you for your consideration various about my patch.

 As you say, I also think that there is no benefit to the existing HTTP
 request processing even if the verification of the IP address by my patch
 was stricter.
 However, I think that this function is useable from a place other than the
 core, as for example plugins. Also I think that it will become very user-
 friendly feature if "WP_Http::is_ip_address()" will be able to strictly
 the IP validation.

 As another approach, I thought that divides the verification processing by
 adding a toggle of strict verification to the second argument of the
 function. But it just would simply become a redundant function.

 I suggested to the ticket because I thought that might be there are happy
 to some people in the case that became to be able to complete IP address
 verification in this function.

 Thank you,

--
Ticket URL: <https://core.trac.wordpress.org/ticket/34538#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list