245425 2022-09-20 07:09:13 -0700 REGRESSION (Safari 16): Service worker clientId is empty for web worker fetches 2023-04-04 20:09:45 -0700 1 1 1 Unclassified WebKit Service Workers Other All iOS 16 RESOLVED FIXED https://github.com/web-platform-tests/wpt/pull/36068 InRadar P2 Critical --- 254938 1 ashley youennf achristensen ahmad.saleem792 apple-rth ap bfulgham cdumez webkit-bug-importer youennf oldest_to_newest 1899806 0 ashley 2022-09-20 07:09:13 -0700 Safari 16 has a regression where the Service Worker fetch event for a web worker has an empty clientId. It works correctly in Chrome and in Safari 15. Minimal repro: https://downloads.scirra.com/labs/swworker/ Load the page. Reload it so the SW becomes active. In its fetch event it logs every fetch with its URL and clientId. Open dev tools for the service worker to see these logs. In Safari 16+ the clientId is empty for the worker fetches. In Safari 15 and Chrome it sets the correct clientId. This breaks a paid-for feature (Remote Preview) in our commercial web app Construct (www.construct.net). Please fix it ASAP. 1899985 1 ap 2022-09-20 18:29:26 -0700 I only see "Hello world from worker.js" in Safari 15.6.1, so this test case doesn't confirm that this is a regression. And I'm getting the same output from STP 153. 1900083 2 ashley 2022-09-21 04:43:25 -0700 I did check iOS 15.7 and it was working there - don't forget to open the console for the Service Worker, it's easy to get mixed up between consoles... 1900084 3 ashley 2022-09-21 04:44:22 -0700 (and don't forget the reload step to make sure the SW is active and actually intercepting fetches) 1900951 4 ap 2022-09-24 16:55:48 -0700 Yes, I wasn't looking at the service worker console, thank you for the clarification. I could confirm with STP, and it worked correctly with Safari 15.6.1. Didn't try Safari 16, but it's known to be broken there per your report. 1900952 5 webkit-bug-importer 2022-09-24 16:56:00 -0700 <rdar://problem/100368228> 1901107 6 youennf 2022-09-26 00:53:01 -0700 Pull request: https://github.com/WebKit/WebKit/pull/4696 1901112 7 youennf 2022-09-26 01:07:35 -0700 Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/36068 1901699 8 ews-feeder 2022-09-28 00:24:06 -0700 Committed 254944@main (6facf7d89d16): <https://commits.webkit.org/254944@main> Reviewed commits have been landed. Closing PR #4696 and removing active labels. 1901854 9 cdumez 2022-09-28 13:24:31 -0700 Reverted by https://github.com/WebKit/WebKit/pull/4801 1901855 10 cdumez 2022-09-28 13:24:34 -0700 Re-opening for pull request https://github.com/WebKit/WebKit/pull/4801 1901858 11 ews-feeder 2022-09-28 13:30:09 -0700 Committed 254966@main (828a55516a7e): <https://commits.webkit.org/254966@main> Reviewed commits have been landed. Closing PR #4801 and removing active labels. 1901879 12 ap 2022-09-28 15:18:05 -0700 So, reopening. 1925813 13 youennf 2023-01-16 06:49:42 -0800 Pull request: https://github.com/WebKit/WebKit/pull/8694 1926272 14 ews-feeder 2023-01-18 08:50:28 -0800 Committed 259031@main (599d00ac6774): <https://commits.webkit.org/259031@main> Reviewed commits have been landed. Closing PR #8694 and removing active labels.