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.

Sign up for GitHub

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

Jump to bottom

;jsessionid is not removed from url #1670

Closed
anonymous-matomo-user opened this issue Sep 3, 2010 · 3 comments
Closed

;jsessionid is not removed from url #1670

anonymous-matomo-user opened this issue Sep 3, 2010 · 3 comments
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.

Comments

@anonymous-matomo-user
Copy link

Piwik uses parse_url to split the url, then the excluded parameters are removed from the query-Result.

The problem is, that the ;jsessionid=... is still in the path-Result because it is separated by ; instead of ?
Example url: http://www.example.com/some/site.jsp;jsessionid=1234?param1=value&param2=value

What's the best way to resolve this problem? Just replace/extend the parse_url function?
@robocoder
Copy link
Contributor

It looks like the examples from #519 didn't make it into #41 / [2023]. We're also missing filters for ASP.net and ColdFusion.

JSP's jsessionid and ASP.net's variants are special cases which can be added to core/Tracker/Action.php's excludeParametersFromUrl().

@robocoder
Copy link
Contributor

See also #1388. (This was also mentioned in #41, but didn't make it into [2023].)

@anonymous-matomo-user anonymous-matomo-user added this to the Future releases milestone Jul 8, 2014
@mattab mattab modified the milestones: Long term, Mid term Dec 23, 2015
@mattab mattab modified the milestones: Long term, Mid term Dec 5, 2016
@sgiehl
Copy link
Member

sgiehl commented Nov 18, 2017

This was fixed in 2.16.1. See https://github.com/piwik/piwik/blob/3.x-dev/CHANGELOG.md#new-features-4

@sgiehl sgiehl closed this as completed Nov 18, 2017
@mattab mattab added the not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. label Dec 5, 2017
@innocraft-automation innocraft-automation removed this from the Backlog (Help wanted) milestone Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement For new feature suggestions that enhance Matomo's capabilities or add a new report, new API etc. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@mattab @robocoder @sgiehl @anonymous-matomo-user @innocraft-automation