When the Allow the cookie to be set for all sites on the same domain. setting is enabled and the drupalSettings.path.baseUrl contains a path like /example, the cookie is set for ONLY that page. This causes the popup to appear on other pages that don't contain that path and the user will need to consent for each page.

CommentFileSizeAuthor
#10 eu_cookie_compliance-allow-cookie-to-be-set-for-all-sites-in-the-domain-when-baseurl-contains-a-path-other-than-home-3071445-10-7.x.patch838 bytessvenryen
#2 eu_cookie_compliance-3071445.patch827 bytesjeremypeter

Comments

jeremypeter created an issue. See original summary.

jeremypeter’s picture

jeremypeter commented
Status: Active » Needs review
StatusFileSize
new eu_cookie_compliance-3071445.patch827 bytes

Status: Needs review » Needs work

The last submitted patch, 2: eu_cookie_compliance-3071445.patch, failed testing. View results

jeremypeter’s picture

jeremypeter commented
Status: Needs work » Needs review
svenryen’s picture

svenryen commented

Thanks, looks good.

jeremypeter’s picture

jeremypeter commented
Status: Needs review » Reviewed & tested by the community
svenryen’s picture

svenryen commented

I don't think you can review your own patch :)

svenryen’s picture

svenryen commented
Status: Reviewed & tested by the community » Needs review

  • jeremypeter authored 79e5840 on 8.x-1.x 
    Issue #3071445 by jeremypeter: Allow cookie to be set for all sites in...
svenryen’s picture

svenryen commented
Status: Needs review » Fixed
StatusFileSize
new eu_cookie_compliance-allow-cookie-to-be-set-for-all-sites-in-the-domain-when-baseurl-contains-a-path-other-than-home-3071445-10-7.x.patch838 bytes

Thanks for catching this. Drupal 7 was also affected. Here's a patch.

  • svenryen authored 16748d5 on 7.x-1.x 
    Issue #3071445 by jeremypeter, svenryen: Allow cookie to be set for all...

Status: Fixed » Closed (fixed)

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