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

Visitors in Real-time plugin: entity codes showed #4018

Closed
anonymous-matomo-user opened this issue Jun 21, 2013 · 1 comment · Fixed by #13928
Closed

Visitors in Real-time plugin: entity codes showed #4018

anonymous-matomo-user opened this issue Jun 21, 2013 · 1 comment · Fixed by #13928
Assignees
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.

Comments

@anonymous-matomo-user
Copy link

In this plugin sometimes some html entity code appear, like the following:

"from Google - "santa caterina d'alessandria iconografia" # 12"

piwik should show

"from Google - "santa caterina d'alessandria iconografia" # 12"
@mattab
Copy link
Member

mattab commented Mar 16, 2014

Note: Let's be careful not to add extra decode as to not open to xss. Maybe we can decode the keyword further before storing it in the log_visit table? it would be nice to reproduce the issue first (is it there on the demo?) before we know how to solve it.

@anonymous-matomo-user anonymous-matomo-user added this to the 2.x - The Great Piwik 2.x Backlog milestone Jul 8, 2014
@mattab mattab removed P: low labels Aug 3, 2014
@mattab mattab added Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. and removed Bug For errors / faults / flaws / inconsistencies etc. labels Oct 11, 2014
@mattab mattab modified the milestones: Long term, Mid term Dec 5, 2016
@sgiehl sgiehl self-assigned this Jan 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants