Bug 275890 – REGRESSION(276197@main): [GTK] Whatsapp Web QR code doesn't load

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Haider Salman]

Created attachment 471746 [details]
demonstration of the spinner

When not logged into web.whatsapp.com the QR code which is supposed to appear on the site doesnt load and instead displays a spinner 

this issue was fixed before [here](https://bugs.webkit.org/show_bug.cgi?id=203404) but it has reappeared with the reason not being the same so I opened this new bug report

[Haider Salman]

Michael Catanzaro I tried it with regular epiphany which is webkitgtk version 2.44.2 and the QR code works fine there

[Michael Catanzaro]

OK, I can reproduce.

(In reply to Haider Salman from comment #1)
> Michael Catanzaro I tried it with regular epiphany which is webkitgtk
> version 2.44.2 and the QR code works fine there

Great, that's important information. Now I can attempt to bisect it, so it's drastically more likely to be fixed.

[Haider Salman]

thats great to hear good luck 👍

[Michael Catanzaro]

This broke in 276197@main "Adding layout tests for postMessage CryptoKey for ServiceWorkers" which has a misleading commit message

[Michael Catanzaro]

Guess: maybe the default key doesn't work anymore?

What is this master key? Why would an application want to override it? Do we need to expose API to allow applications to configure it? Surely WebCrypto is not just expected to fail by default now?

[Michael Catanzaro]

So if this is preventing users from logging into WhatsApp, we should revert it now and ask questions later. Too bad I didn't do that in June, when it would have been easier, because there are conflicts. I don't care enough about WhatsApps to sort through this myself; anybody else should feel free.

[Nitin Mahendru]

<rdar://problem/136807212>

[Nitin Mahendru]

Pull request: https://github.com/WebKit/WebKit/pull/34348

[Nitin Mahendru]

Is it possible to verify that your problem is solved with this patch ? IIUC it should.
@Michael, @Haider ?

[Michael Catanzaro]

I'll test soon.

[Michael Catanzaro]

(In reply to Nitin Mahendru from comment #9)
> Is it possible to verify that your problem is solved with this patch ? IIUC
> it should.
> @Michael, @Haider ?

Confirmed, your patch works. Thanks!

[EWS]

Committed 284377@main (0a79bb9adb4c): <https://commits.webkit.org/284377@main>

Reviewed commits have been landed. Closing PR #34348 and removing active labels.

[Adrian Perez]

*** Bug 280582 has been marked as a duplicate of this bug. ***