On sites that use the EU Cookie Compliance module we should not track users that opted out of cookies/tracking.
Additionally, it should be possible to adapt this logic in custom code so that sites not using that module could add their own logic.

Issue fork searchstax-3569651

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:

About issue forks

Comments

drunken monkey created an issue. See original summary.

drunken monkey opened merge request !72

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Active » Needs work

I created a merge request that takes care of the integration with the EU Cookie Compliance module. I also added test coverage, but test is apparently a bit flaky on GitLab CI (works perfectly fine locally, but that’s no use of course).

Also, Drupal 8 pipelines are broken in HEAD, so will have to fix that first before we’ll be able to merge this one.

Finally, the interface for custom modules to influence this behavior is still missing. Will have to add that and its test coverage, too.

  • drunken monkey committed fe7d1d84 on 1.x 
    feat: #3569651 Added integration with the EU Cookie Compliance module....
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs work » Needs review

Merged to HEAD, will be tested there.

drunken monkey opened merge request !78

  • drunken monkey committed 9c36e54a on 1.x 
    fix: #3569651 Fixed two bugs in the EU Cookie Compliance integration....
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Fixed two bugs that were uncovered:

  1. When rejecting tracking, even though there was no tracking event sent the ms_visitorid cookie was still set.
  2. The link to the EU Cookie Compliance settings page on our settings page was missing its URL.
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs review » Fixed

This has now been confirmed to work as intended and released as part of version 1.10.0.

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.