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

spike after updating matomo to 3.13 #15205

Closed
Risingson opened this issue Nov 28, 2019 · 13 comments
Closed

spike after updating matomo to 3.13 #15205

Risingson opened this issue Nov 28, 2019 · 13 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@Risingson
Copy link

Risingson commented Nov 28, 2019

image
seems like actions of a single user are counted as unique visits

@tsteur
Copy link
Member

tsteur commented Nov 28, 2019

I would suspect this is due to #14360 maybe? Are you using the userId feature?

@Risingson
Copy link
Author

Risingson commented Nov 28, 2019

You are right i am using the userID on mobile apps. I am sure the mentioned change is related. But still: Triggering an action should not count a unique visit, should it?

@tsteur
Copy link
Member

tsteur commented Nov 28, 2019

Not sure what you mean? Triggering an action should indeed not create a new unique visit and it doesn't unless your visitors were to regenerate the visitorId on every request somehow. Not sure how you embed the tracking code into your mobile app? It seems your using maybe a different visitorId each time

@Risingson
Copy link
Author

Risingson commented Nov 28, 2019

I do not intentionally create a new visitorId. I can provide code if you want me to.

@Risingson
Copy link
Author

Risingson commented Nov 28, 2019

These are 1 initial visit + 2 action calls of the same session counting as 3 visits obviousely.
image

Normally it would look like this and count one unique visit:
image

the only difference between the tracking code implementations of the 2 above is that the mobile one sets a userID.

@Risingson
Copy link
Author

Any news on this ?

@tsteur
Copy link
Member

tsteur commented Dec 3, 2019

It really depends how you track your mobile app. Are you using one of our android or iOS SDKs? Then I would create an issue and ask there. If you implemented it yourself, then maybe have a look at the Tracking API docs: https://developer.matomo.org/api-reference/tracking-api . Maybe you need to set the _id parameter

@Risingson
Copy link
Author

So this is intentional behaviour of a minor release that rips functionality provided up until 3.13.
Nice one.

@mattab
Copy link
Member

mattab commented Dec 10, 2019

@Risingson Are you using the official Matomo Android SDK? If so, it would be a bug in the SDK and it would be great if you can open a bug report there, or we can move this report to the Android SDK project?
also refs #15242

@Risingson
Copy link
Author

Risingson commented Dec 10, 2019

I am not using the Android SDK.
It is a mobile app using webviews (html+js>phonegap). I can send the tracking code if you wish.
As stated the only difference within the tracking code between the browser app and the mobile app is that mobile sets a device ID as user ID. _paq.push(['setUserId', device.uuid]);. Such an UserId is visible in the example screenshot above:
image
Still it looks like as if each action is causing a unique visit count.

@tsteur
Copy link
Member

tsteur commented Dec 10, 2019

Maybe there are issues with cookies and it always regenerates a random visitorId. If it uses the regular JS tracking code in the webview, it should still identify each visitor based on IP and other critieria (although in a mobile app a lot of the criteria be the same). Can you check if you maybe have trust_visitors_cookies enabled in your config file? You may have enabled it since otherwise some visitors are grouped together even though they are different visitors in webview mobile apps.

What you likely want to do is call _paq.push(['setVisitorId', device.uuid]). The unique visitor ID, must be a 16 characters hexadecimal string.

tsteur added a commit that referenced this issue Dec 10, 2019
refs #15205

Since we no longer use the UserId as VisitorId I reckon this needs changing. I haven't tested anything but it seems logical to me. Otherwise there might be issues where no visitorId is set but a userId and then it always creates new visits.
@tsteur
Copy link
Member

tsteur commented Dec 10, 2019

Also created a PR that might help: #15258

@tsteur tsteur reopened this Dec 10, 2019
@tsteur tsteur added this to the 3.13.1 milestone Dec 10, 2019
@Risingson
Copy link
Author

Risingson commented Dec 14, 2019

Glad to hear you are trying to find a way to fix this. I cannot force people to update the apps on their phones...

diosmosis pushed a commit that referenced this issue Dec 15, 2019
refs #15205

Since we no longer use the UserId as VisitorId I reckon this needs changing. I haven't tested anything but it seems logical to me. Otherwise there might be issues where no visitorId is set but a userId and then it always creates new visits.
@tsteur tsteur closed this as completed Jan 8, 2020
@tsteur tsteur added the Bug For errors / faults / flaws / inconsistencies etc. label Jan 8, 2020
jonasgrilleres pushed a commit to 1024pix/pix-analytics that referenced this issue Sep 22, 2020
refs matomo-org#15205

Since we no longer use the UserId as VisitorId I reckon this needs changing. I haven't tested anything but it seems logical to me. Otherwise there might be issues where no visitorId is set but a userId and then it always creates new visits.
jbuget pushed a commit to 1024pix/pix-analytics that referenced this issue Sep 26, 2020
refs matomo-org#15205

Since we no longer use the UserId as VisitorId I reckon this needs changing. I haven't tested anything but it seems logical to me. Otherwise there might be issues where no visitorId is set but a userId and then it always creates new visits.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

3 participants