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

Jump to bottom

Anonymous user settings results in an error when site no longer exists #13423

Closed
tsteur opened this issue Sep 13, 2018 · 0 comments · Fixed by #13433
Closed

Anonymous user settings results in an error when site no longer exists #13423

tsteur opened this issue Sep 13, 2018 · 0 comments · Fixed by #13433
Assignees
@tsteur
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
3.6.1

Comments

@tsteur
Copy link
Member

tsteur commented Sep 13, 2018 •
edited

I visit https://apache.matomo/index.php?module=UsersManager&action=anonymousSettings&idSite=1&period=day&date=yesterday and it says:

An unexpected website was found in the request: website id was set to '10' .

The GET request is initially correct
image

but causes a problem in UsersManager Controller:

image

The anonymous user used to have access to those sites...
image

Not sure if access was maybe not removed when the site was deleted, whether data was deleted manually, or if it is a regression etc.

We'll probably need a try /catch around this block.
@tsteur tsteur added the Bug For errors / faults / flaws / inconsistencies etc. label Sep 13, 2018
@tsteur tsteur added this to the 3.6.1 milestone Sep 13, 2018
@tsteur tsteur self-assigned this Sep 17, 2018
@tsteur tsteur mentioned this issue Sep 17, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsteur tsteur

Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
3.6.1
Development

Successfully merging a pull request may close this issue.

Return access entries only for sites that still exist matomo-org/matomo
1 participant
@tsteur