Log in / log out problems in Chrome

bug still exists

We also detect the problem in Chrome, in IE, in Opera.
Drupal core 9.5.5.
Both in login or logout the system runs empty. To resolve the user must click on a link and quickly manages to enter.
Also note an excessive slowdown of the system when loading pages, views ...
Is it possible to help us to solve the problem?
Thank you

Same issue too...

I'm having the same problem +1

Sorry guys, I don't have the time to write a patch currently. If anyone has the time, please do it, including tests and I will review it quickly!

Created a patch for the issue mentioned, please review.
Thank You.

Well that doesn't really help. It's just what has been written in the issue summary. What we need is a clean fix, explanation why it was used before, but should (perhaps?) not be used anymore and what are the consequences.

The result of #17 is just commented out code with unclear results from my perspective. Anyway I'll prepare a MR now which removes these lines to be used as workaround and have a look if I can find a blame entry where it was added ...

These lines were added here: 🐛 Service Worker caches admin toolbar and contextual links Fixed

So it seems this is a regression. I'll ask the users from that issue to have a look please.

I'm wondering if perhaps the (double) quotes may cause this?
'"storage"' vs, 'storage' as string value?

Could someone try that?

This patch https://git.drupalcode.org/project/pwa/-/merge_requests/50.diff works for me

Thanks @Kris77 interesting to hear! That sounds good and if it also works for others, we should have a deeper look at the headers sent and if they work as expected (clearing those caches)!

Verified the patch MR!50 it resolve the issue of user login and logout.

+RTBC

I don't face this issue when user logout but face the exactly issue when user login.
The patch MR!50 is solved my problem when user login.

Could someone please compare and debug the mentioned Clear-Site-Data header with and without this patch?

We need to be totally sure that this is the correct solution, before merging this. I don't have much time currently, sorry.

I couldn't recreate this issue under windows using Chrome Version 113.0.5672.93 (Official Build) (64-Bit) and Olivero 10.0.9.

Although the patch in MR 50 makes sense... @Anybody, how do you want to proceed here?

@Grevil: Please see #22 could you check these header values on the login page please with and without the patch?

My guess, which the patch is based on, was, that the quotes are unexpectedly output with the string, which breaks the functionality.

Hi folks, thanks for working on this.

I've tested MR!50 - it works!

+1 for RTBC.

Headers without patch:

Headers with patch applied:

@Grevil: Thank you sooooo much, so my assumption was correct :D Whao!

Let's merge it!

Anybody → closed merge request !49

Cherry-picked to 2.x and created 8.x-1.7 and 2.0.0-rc2 with this fix included! Thank you all!

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