Closed (fixed)
Project:
COOKiES Consent Management
Version:
1.2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Feb 2023 at 10:12 UTC
Updated:
19 Mar 2024 at 14:09 UTC
Jump to comment: Most recent

Comments
Comment #2
anybodyThanks for the report. I'd also expect "0" (session duration) to work. We should add a test for that.
@Grevil will have a look and if there are any questions, contact me.
In the meantime, anyone may provide a MR or details to speed things up.
Comment #3
grevil commentedWow, that is really weird...
It even says in the description, "Set to 0 to expire with browser close."
Seems this feature broke at some point, and we simply had no test for this...
But yes, I can reproduce this issue.
Comment #4
grevil commentedComment #5
anybodyProbably this is the wrong line: https://git.drupalcode.org/issue/cookies-3343952/-/blob/3343952-expirati...
Guess the empty check is incorrect.
An cookie expiration of 0 (session duration) should be fine. But please test carefully.
Comment #6
anybodyComment #8
grevil commentedFixed! Works perfectly fine now!
Please review!
Comment #9
anybodyGreat work!! :)
Comment #10
anybody