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

GET Parameter with square brackets is not excluded properly. #3201

Closed
anonymous-matomo-user opened this issue Jun 6, 2012 · 4 comments
Closed
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@anonymous-matomo-user
Copy link

I have get parameters with square brackets "[]" in it. I can not blacklist those parameter in the admin interface.

Chrome is URL encoding the brackets:
http:://MYDOMAIN/sompage?von%5Bvalue%5D%5Bdate%5D=01.01.2012&bis%5Bvalue%5D%5Bdate%5D=31.12.2012

FF is displaying it:
http:://MYDOMAIN/sompage?von[value][date]=&bis[value][date]=

I tried to enter both variants at the admin page
http://PIWIKDOMAIN/index.php?module=SitesManager&action=index&idSite=1&period=day&date=yesterday
but this is not working. The encoded version is not saved. The other version is saved but in the URL List the parameter still shows up and is not ignored.

@catchin
Copy link

catchin commented Jul 4, 2012

for a fix, see the latest patch for #3187

@mattab
Copy link
Member

mattab commented Jul 19, 2012

Thanks for report and fix

@mattab
Copy link
Member

mattab commented Aug 3, 2012

(In [6659]) Fixes #3187 Handle java style matrix URLs
Fixes #3201 GET Parameter with square brackets can now be excluded properly

Thanks catchin for the patches!!
Please consider contributing further patches if you can :)

@sgiehl
Copy link
Member

sgiehl commented Aug 3, 2012

(In [refs #3227, #3107, #3201 changed phpunit test according to changes in 6659)

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.
Projects
None yet
Development

No branches or pull requests

4 participants