Problem/Motivation

Matomo has a setVisitorCookieTimeout option that can be used to customize the duration of the cookie.
It would be useful if it could be set from the config of this module.

Proposed resolution

We could add a new field in Privacy on /admin/config/system/matomo.

Issue fork matomo-3376588

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

prudloff created an issue. See original summary.

prudloff’s picture

Status: Active » Needs review
prudloff’s picture

Rebased against 2.0.x.

The failing phpunit test seems to be unrelated: #3511606: MatomoBasicTest::testMatomoPageVisibility() fails in merge requests

arousseau’s picture

Status: Needs review » Reviewed & tested by the community

This is working correctly and the "_pk_id" cookie duration follows the value set in the "Cookie duration" field, added to the settings form.
The "_pk_sess" cookie duration is left untouched as it should (30 min default).

prudloff’s picture

Issue tags: +Privacy improvements
grimreaper’s picture

Hello,

Thanks for the issue and MR.

Would it be possible to add a test please?

prudloff’s picture

I added a test.

grimreaper’s picture

Status: Reviewed & tested by the community » Needs review

Thanks, one question about existing installations in code review.

grimreaper’s picture

Assigned: Unassigned » grimreaper

  • grimreaper committed 939dbf56 on 2.0.x authored by prudloff
    feat: #3376588 Add support for "setVisitorCookieTimeout"
    
    By: prudloff...
grimreaper’s picture

Assigned: grimreaper » Unassigned
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.