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

[POC] Obfuscate tracking parameters #14211

Closed
wants to merge 3 commits into from
Closed

[POC] Obfuscate tracking parameters #14211

wants to merge 3 commits into from

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Mar 16, 2019

Even when renaming piwik.php (or matomo.php) some adblockers still seem to block some tracking requests based on the url parameters sent with the request.

This implementation will make it possible to obfuscate all request parameters by doing a base64 on the request string.

Not sure if it might make sense to even enable that by default if it works as expected.

refs #7364, #14207

@sgiehl sgiehl added the RFC Indicates the issue is a request for comments where the author is looking for feedback. label Mar 16, 2019
@tsteur
Copy link
Member

tsteur commented Mar 16, 2019

wouldn't you just send it as POST maybe if someone really wanted to avoid it? Of course you can't replay maybe but would probably work otherwise. Not sure if it has high prio right now

@sgiehl
Copy link
Member Author

sgiehl commented Mar 16, 2019

Correct, POST would be another way to avoid a parameter blocking, but as you mentioned it wouldn't work if replaying logs is required at some point.

Actually this was just a simple idea that came up to my mind when reading some issues and forum posts. Thought it might be worth to create a simple draft for it so we can consider using it at some point if usefull...

@fdellwing
Copy link
Contributor

I kind of like this idea. Just to be sure, it will not be .php?obfuscate=abc but .php?abc?

@kdekooter
Copy link

Try to get rid of the .php extension as well. It is a red flag for blockers.

@tsteur
Copy link
Member

tsteur commented Apr 11, 2019

@sgiehl I think we can maybe close the issue for now as we don't really want to work around ad blockers / privacy lists in such a way at this stage.

@negreanucalin
Copy link

Maybe it would be a good idea to take this POC into account and maybe define some requirements, I have made the the mentioned post above with a temporary workaround.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFC Indicates the issue is a request for comments where the author is looking for feedback.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants