[wp-trac] [WordPress Trac] #40871: Remove IP Address Anonymization From WP_Community_Events

[WordPress Trac]

#40871: Remove IP Address Anonymization From WP_Community_Events
----------------------------+------------------------------
 Reporter:  iandunn         |       Owner:
     Type:  enhancement     |      Status:  closed
 Priority:  normal          |   Milestone:  Awaiting Review
Component:  Administration  |     Version:  trunk
 Severity:  normal          |  Resolution:  maybelater
 Keywords:  has-patch       |     Focuses:
----------------------------+------------------------------
Changes (by iandunn):

 * status:  new => closed
 * resolution:   => maybelater


Comment:

 Doh, it looks like I misdiagnosed the problem. The problem was actually
 that the ip2location lookup for the IP returned the wrong result. That's
 bound to happen from time to time, since IP geolocation is only a rough
 estimate, and changes often. I notified ip2location, so that they can fix
 their database.

 It's possible that there will be other cases in the future where the
 anonymization will distort the results, but we can cross that bridge when
 we have specific examples to test.

 If that does happen, some of
 [https://wordpress.slack.com/archives/C02RQBWTW/p1496169353382618 today's
 discussion on Slack might be helpful]. It seems like using a smaller
 `$netmask`, like `255.255.255.248`, might work well to keep the IP
 partially anonymous, but reduce the chance of incorrect results.

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