¿What is this cookie cookie-agreed-version? This cookie is send without any consent of the user, it seems a bug.

and thanks for this great module!

CommentFileSizeAuthor
#2 Screenshot from 2021-01-29 13-39-32.png22.31 KBcarlitus

Issue fork eu_cookie_compliance-3195373

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

Carlitus created an issue. See original summary.

carlitus’s picture

carlitus commented
Issue summary: View changes
StatusFileSize
new Screenshot from 2021-01-29 13-39-32.png22.31 KB
f0ns’s picture

f0ns commented

I see the same behaviours happening, the cookie is written from the eu_cookie_compliance.js script. Disabling this script before consent is given is of course not an option as it is used to fire the Cookie compliance functionality.

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Related issues: +#3193826: Allow never setting policy version cookie
steffenr’s picture

steffenr
Primary language German
Location Bremen
commented

We had the same issue with one of our clients.

In my opinion it would make sense, that the cookie-agreed-version should only be set, if the user really agreed to the cookie consent. I'll work on a merge request for this change.
In the attached MR i fixed the issue - i also had to make small change the handling of the closed status of the popup.

Maybe it would better to make this new behaviour configurable - what do you think?

SteffenR opened merge request !8

svenryen’s picture

svenryen commented

Looks good. I'll apply the MR and test later.

svenryen’s picture

svenryen commented
Status: Active » Needs review

  • SteffenR committed 7b94de1 on 8.x-1.x 
    #3195373: fix handling of popup close status for cookie-agreed 0
  • SteffenR committed e79a815 on 8.x-1.x
    #3195373: only set cookie-agreed-version, if user agreed to cookies
svenryen’s picture

svenryen commented
Status: Needs review » Fixed

Thanks for the patch. I've pushed your fix!

Status: Fixed » Closed (fixed)

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

carlitus’s picture

carlitus commented