Navigation Menu

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

csv export for VisitsSummary.getVisits with idSite=all and non-superuser not working #14362

Closed
Findus23 opened this issue Apr 19, 2019 · 2 comments · Fixed by #15584
Closed

csv export for VisitsSummary.getVisits with idSite=all and non-superuser not working #14362

Findus23 opened this issue Apr 19, 2019 · 2 comments · Fixed by #15584
Assignees
@sgiehl
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
4.0.0

Comments

@Findus23
Copy link
Member

Reported twice in https://forum.matomo.org/t/matomo-api-to-fetch-results-for-idsite-all-in-csv-xls-or-tsv-format-not-working-for-normal-users/31663 and reproduced on my Matomo instance

Requesting the VisitsSummary.getVisits API when

  • the user of the token_auth is not a superuser
  • idSite=all is requested
  • and format=csv (or xls)

then the following error appears:

Error: The parameter 'idSite=' is missing from the request.

All other requests seem to work fine (e.g. format=json)
@Findus23 Findus23 added the Bug For errors / faults / flaws / inconsistencies etc. label Apr 19, 2019
tsteur added a commit that referenced this issue Apr 22, 2019
@tsteur
csv export for VisitsSummary.getVisits with idSite=all and non-superu…
abbd69e 
…ser not working

fix  #14362
@tsteur
Copy link
Member

tsteur commented Apr 22, 2019

I can't really test it right now @Findus23 could you maybe try https://github.com/matomo-org/matomo/compare/3.x-dev...14362?quick_pull=1 ?

@mattab
Copy link
Member

mattab commented Jan 21, 2020

@Findus23 are you able to test the patch from the last comment above?

@mattab mattab added this to the 4.0.0 milestone Jan 21, 2020
@sgiehl sgiehl self-assigned this Feb 17, 2020
@sgiehl sgiehl mentioned this issue Feb 17, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sgiehl sgiehl

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

Successfully merging a pull request may close this issue.

Fix CSV export for idSite=all matomo-org/matomo
4 participants
@tsteur @mattab @sgiehl @Findus23