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

rememberConsentGiven only lasts for session #15321

Closed
wlabarron opened this issue Dec 29, 2019 · 2 comments · Fixed by #15324
Closed

rememberConsentGiven only lasts for session #15321

wlabarron opened this issue Dec 29, 2019 · 2 comments · Fixed by #15324
Labels
Bug For errors / faults / flaws / inconsistencies etc.

Comments

@wlabarron
Copy link

wlabarron commented Dec 29, 2019 •
edited

Similar to #14487, the mtm_consent cookie, added when calling _paq.push(['rememberConsentGiven']) and used to remember that consent to track was given, lasts for session instead of 30 years.

mtm_consent_removed lasts for 30 years.
@wlabarron wlabarron changed the title _paq.push(['rememberConsentGiven']); only lasts for session rememberConsentGiven only lasts for session Dec 29, 2019
@wlabarron
Copy link
Author

wlabarron commented Dec 29, 2019 •
edited

I see in the documentation that I can pass a number of hours to expire after, and I'm currently using that as a workaround. The docs do say though that the cookie should last for longer than session:

By default, the cookie and consent will be remembered forever. It is possible to define an optional expiry period for your user consent by calling: _paq.push(['rememberConsentGiven', optionallyExpireConsentInHours]).

@tsteur tsteur added the Bug For errors / faults / flaws / inconsistencies etc. label Dec 29, 2019
@tsteur
Copy link
Member

tsteur commented Dec 29, 2019

I suppose this should last by default also for 30 years? Or we need to change the documentation to say the number of hours is required.

@tsteur tsteur mentioned this issue Dec 29, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remember consent given by default for 30 years matomo-org/matomo
2 participants
@tsteur @wlabarron