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

setCustomRequestProcessing does not get"Page Performance" parameters passed #18736

Closed
McSodbrenner opened this issue Feb 3, 2022 · 1 comment
Closed

setCustomRequestProcessing does not get"Page Performance" parameters passed #18736

McSodbrenner opened this issue Feb 3, 2022 · 1 comment
Labels
answered For when a question was asked and we referred to forum or answered it.

Comments

@McSodbrenner
Copy link

Current Behavior

I rewrite the GET parameters of the tracking request via setCustomRequestProcessing(). However, I noticed that the page performance parameters (https://developer.matomo.org/api-reference/tracking-api#optional-page-performancehttpsmatomoorgfaqhow-tohow-do-i-see-page-performance-reports-info) are not included there. They are added to the parameter string at a later time.

Expected Behavior

The description on https://developer.matomo.org/api-reference/tracking-javascript for setCustomRequestProcessing() I understand so that the function is called directly before sending the request. Therefore, I expect all parameters to be included in it.

Your Environment

  • Matomo-Version: 4.7.1
  • PHP-Version: 7.2.23
  • Browser: Chrome 97
@McSodbrenner McSodbrenner added the Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. label Feb 3, 2022
@sgiehl
Copy link
Member

sgiehl commented Feb 4, 2022

@McSodbrenner The method configCustomRequestContentProcessing is called last when generating the request.
But the page performance parameters might not yet be available when the first tracking request is fired. If that is the case the parameters will automatically be included in the next request (on the same page). If you want to have all parameters in one request you might need to wait until the page load event is finished.

@sgiehl sgiehl closed this as completed Feb 4, 2022
@sgiehl sgiehl added answered For when a question was asked and we referred to forum or answered it. and removed Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. labels Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
answered For when a question was asked and we referred to forum or answered it.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sgiehl @McSodbrenner