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

Anonymize IP should anonymize IPv6 addresses #3710

Closed
anonymous-piwik-user opened this issue Jan 25, 2013 · 7 comments

Comments

Projects
None yet
3 participants
@anonymous-piwik-user
Copy link

commented Jan 25, 2013

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

This comment has been minimized.

Copy link
Member

commented Feb 7, 2013

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

@anonymous-piwik-user

This comment has been minimized.

Copy link
Author

commented Feb 8, 2013

Attachment:
piwik_anonymizeip.png

@anonymous-piwik-user

This comment has been minimized.

Copy link
Author

commented Feb 8, 2013

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 comment has been minimized.

Copy link
Author

commented Apr 13, 2013

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

This comment has been minimized.

Copy link
Author

commented Sep 3, 2013

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

This comment has been minimized.

Copy link
Author

commented Sep 3, 2013

In 2c378a2: Fix test, simplify

refs #3710

@mattab

This comment has been minimized.

Copy link
Member

commented Oct 16, 2013

Great fix!

@anonymous-piwik-user anonymous-piwik-user added this to the 2.0 - Piwik 2.0 milestone Jul 8, 2014

sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014

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 matomo-org#3710

sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.