273906 2024-05-08 15:19:09 -0700 `navigator.storage.getDirectory()` fails in service workers with an `NotSupportedError: The operation is not supported` 2024-06-23 19:54:23 -0700 1 1 1 Unclassified WebKit Service Workers Safari 17 Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=256712 https://github.com/web-platform-tests/wpt/pull/46690 BrowserCompat, InRadar P2 Normal --- 1 steve youennf ahmad.saleem792 cdumez karlcow sihui_liu webkit-bug-importer youennf oldest_to_newest 2034126 0 steve 2024-05-08 15:19:09 -0700 This is different from https://bugs.webkit.org/show_bug.cgi?id=255458 2034146 1 steve 2024-05-08 16:17:38 -0700 To reproduce: 1. Go to https://opfs-in-service-worker.glitch.me/ in Chrome or Firefox. 2. Open the Console. 3. Note how calls to `navigator.storage.getDirectory()` succeed in the service worker. 4. Go to https://opfs-in-service-worker.glitch.me/ in Safari. 5. Open the Console. 6. Note how calls to `navigator.storage.getDirectory()` succeed in the main thread, web worker, but fail in a service worker with an `NotSupportedError`. 2034164 2 ahmad.saleem792 2024-05-08 17:18:07 -0700 @Jiwen - is it recent regression and use to work earlier? 2034165 3 webkit-bug-importer 2024-05-08 17:18:17 -0700 <rdar://problem/127775672> 2034168 4 steve 2024-05-08 17:21:15 -0700 Hi @Ahmad, this is the first time we attempted to use this. Not sure it's a regression. 2034215 5 karlcow 2024-05-08 22:25:37 -0700 Probably Youenn might know about it. 2034920 6 youennf 2024-05-13 07:01:12 -0700 We are exposing StorageManager to service workers but are not setting the service worker StorageProvider. We should probably set the StorageProvider or alternatively disable StorageManager in service worker contexts. 2038312 7 steve 2024-05-28 15:52:21 -0700 Hey folks, thanks for taking a look at this. Supporting OPFS in service worker helps a lot for us to better managing large file caching using local storage. It's quite useful feature overall. 2040279 8 youennf 2024-06-07 03:03:17 -0700 Pull request: https://github.com/WebKit/WebKit/pull/29615 2040786 9 youennf 2024-06-11 05:44:24 -0700 Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/46690 2040799 10 ews-feeder 2024-06-11 06:54:28 -0700 Committed 279908@main (c47f4cabd53d): <https://commits.webkit.org/279908@main> Reviewed commits have been landed. Closing PR #29615 and removing active labels. 2042696 11 steve 2024-06-23 19:54:23 -0700 Thanks for fixing this. Do we know which Safari version can be expect this to land?