Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Anonymize IP should anonymize IPv6 addresses #3710

Closed
anonymous-piwik-user opened this Issue · 7 comments

3 participants

@anonymous-piwik-user

Hello everybody,

I noticed that the anonimzeIP plugin does not anonymize/mask IPv6 addresses.

The plugin should mask the last 3 bytes of an IP address.
It works well for IPv4 addresses, but not for IPv6 addresses.
The newest (10.0.1) version of piwik was used.

Maybe someone could take a look at this plugin.

Regards,

Sebastian Neef aka gehaxelt

Keywords: Anonymize IP

@mattab
Owner

Can you post a screenshot showing the problem, ie. what is your IP and what is the anonimized IP ?

@anonymous-piwik-user

Hallo,

I'm not connected to the internet via IPv6 so I can't give you the diff between real IPv6 and anonymized IPv6.

I attached a screenshot of IPv4 and IPv6 addresses.

I hope I could help somehow.

Regards,

gehaxelt

@anonymous-piwik-user

This is really a problem, in Germany it is not allowed to store the IP addresses.
For IPv4 addresses it is working with the plugin anonymize IP. THis should be extended ASAP to include support for IPv6.

@anonymous-piwik-user

In c072a0e: Extend AnonymizeIP to properly mask IPv6 addresses.

maskLength is now interpreted as "mask level" (0-4) and determines the number of bits to be masked.

Level IPv4 IPv6
0 0 0
1 8 64
2 16 80
3 24 104
4 32 n/a

fixes #3710

@anonymous-piwik-user

In 2c378a2: Fix test, simplify

refs #3710

@mattab
Owner

Great fix!

@anonymous-piwik-user anonymous-piwik-user added this to the 2.0 - Piwik 2.0 milestone
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@halfdan halfdan Extend AnonymizeIP to properly mask IPv6 addresses.
maskLength is now interpreted as "mask level" (0-4) and determines the number of bits to be masked.

Level	IPv4	IPv6
0	0	0
1	8	64
2	16	80
3	24	104
4	32	n/a

fixes #3710
c072a0e
@sabl0r sabl0r referenced this issue from a commit in sabl0r/piwik
@halfdan halfdan Fix test, simplify
refs #3710
2c378a2
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.