Merge pull request #837 from infinum/feature/domain-argument-in-setco…

…okie Support an optional `domain` argument in setCookie helper
infinum · Jul 31, 2024 · 253fa10 · 253fa10
2 parents 298d666 + f878ed1
commit 253fa10
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/scripts/helpers/cookies.js b/scripts/helpers/cookies.js
@@ -20,16 +20,22 @@ export const cookies = {
 	 * cookies.setCookie('gdpr', '2', cookies.setOneDay(), '/');
 	 * ```
 	 */
-	setCookie(key, value, time, path) {
+	setCookie(key, value, time, path, domain) {
 		const expires = new Date();
 		expires.setTime(expires.getTime() + (time));
+
 		let pathValue = '';
+		let domainValue = '';
 
 		if (typeof path !== 'undefined') {
-			pathValue = `path=${path};`;
+			pathValue = `;path=${path}`;
 		}
 
-		document.cookie = `${key}=${value};${pathValue}expires=${expires.toUTCString()}`;
+		if (typeof domain !== 'undefined') {
+			domainValue = `;domain=${domain}`;
+		}
+
+		document.cookie = `${key}=${value}${pathValue}${domainValue};expires=${expires.toUTCString()}`;
 	},
 
 	/**