Fix breakage report race condition.

Config processing can how take one tick after the config ready promise resolves. Waiting for allLoadingFinished should ensure that everything is ready.
duckduckgo · Feb 5, 2025 · 53504de · 53504de
1 parent 8e16d35
commit 53504de
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/shared/js/background/broken-site-report.js b/shared/js/background/broken-site-report.js
@@ -321,6 +321,8 @@ export async function breakageReportForTab({
 export async function sendBreakageReportForCurrentTab({ pixelName, currentTab, category, description, reportFlow }) {
     await settings.ready();
     await tdsStorage.ready('config');
+    // wait for onload callbacks (to ensure that config has been correctly processed)
+    await tdsStorage.config.allLoadingFinished
 
     const tab = currentTab || (await tabManager.getOrRestoreCurrentTab());
     if (!tab) {