Attachment 150413 Details for Bug 33597 – Bug 33597: Get rid of warning about intranet_bib_list

[patch] Bug 33597: Get rid of warning about intranet_bib_list

Bug-33597-Get-rid-of-warning-about-intranetbiblist.patch (text/plain), 1.86 KB, created by Nick Clemens (kidclamp) on 2023-04-28 18:43:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2023-04-28 18:43:11 UTC

Size: 1.86 KB

patch

obsolete

>From 094622ca6c27aa853bcd6296956edb2e32b55c61 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Mon, 24 Apr 2023 10:23:19 +0000
>Subject: [PATCH] Bug 33597: Get rid of warning about intranet_bib_list
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Resolve:
>Cookie âintranet_bib_listâ does not have a proper âSameSiteâ attribute value. Soon, cookies without the âSameSiteâ attribute or with an invalid value will be treated as âLaxâ. This means that the cookie will no longer be sent in third-party contexts. If your application depends on this cookie being available in such contexts, please add the âSameSite=Noneâ attribute to it.
>basket_22.1200022.js:67:65
>
>Test plan:
>Do not apply this patch.
>Logout from staff.
>Check browser console warnings.
>Apply this patch.
>Login and logout again.
>Check warnings again.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> koha-tmpl/intranet-tmpl/prog/js/basket.js | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/basket.js b/koha-tmpl/intranet-tmpl/prog/js/basket.js
>index fa03d434ca..f72d58452b 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/basket.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/basket.js
>@@ -62,9 +62,9 @@ function delCookie(name) {
>     var exp = new Date();
>     exp.setTime(exp.getTime()-1);
>     if(parent.opener){
>-        parent.opener.document.cookie = name + "=null; path=/; expires=" + exp.toGMTString();
>+        parent.opener.document.cookie = name + "=null; SameSite=Lax; path=/; expires=" + exp.toGMTString();
>     } else {
>-        document.cookie = name + "=null; path=/; expires=" + exp.toGMTString();
>+        document.cookie = name + "=null; SameSite=Lax; path=/; expires=" + exp.toGMTString();
>     }
> }
> 
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 33597: 150148 | 150149 | 150150 | 150151 | 150413 | 150414 | 150415