Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When anonimize IP is enabled, consider doing Provider / GeoIP lookup faking the last IP byte to 1 ? #3023

Closed
mattab opened this issue Mar 8, 2012 · 3 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Mar 8, 2012

It is suggested in the forums that major internet providers own the full range of IP.

When IP has only 1 byte anonimized, could we use the assumption that they own the x.y.z.1 IP, and do the provider lookup on this IP, instead of x.y.z.0 which does not work (and is currently disabled)?

It might fail to return the proper Provider in some cases, but for most cases, it should work. Thoughts?

@mattab
Copy link
Member Author

mattab commented May 31, 2012

We should also consider GeoIP which has the same issue, how about replacing bytes by 0 and doing lookup in GeoIP based on that? I have the feeling I had this discussion already...

@anonymous-matomo-user
Copy link

Hi,
this issue is also important to me. Because I don't have the ability to look deep in the code I don't know why this is not possible, but for me the best solution will be

  • getting the provider
  • anonimize the ip afterwards

I saw in the forum that this is maybe due tu privacy reasons, but for me this is not the case. the ip address is not needed to be saved to any database, its just used in the code (ram) and then anonymized directly.

rgds.
Michael

@diosmosis
Copy link
Member

Anonymization works w/ GeoIP, though can become less accurate (depending on number of bytes anonymized).

@mattab mattab added this to the 1.12.x - Piwik 1.12.x milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Development

No branches or pull requests

3 participants