fix: static analysis remediation
- Add error logging to inlineAllResources() catch block (P2) - Logs warning when resource inlining fails instead of silent fail - Updated SEC 22 header comment to reflect change Note: bakeStyles is correctly scoped to URLItem (per-URL), not View-level. The DB column exists but code properly uses urlItem.bakeStyles.
This commit is contained in:
+3
-2
@@ -35,7 +35,7 @@
|
||||
* 19 | 645 | SSR INLINING / injectMetaRefresh | Meta-refresh injection, script stripping
|
||||
* 20 | 663 | CSS IMPORT RESOLVER / @import | Recursive @import resolver (max 3 levels)
|
||||
* 21 | 694 | COMPUTED STYLE BAKING / bake | Inline computed styles for Tizen compat
|
||||
* 22 | 777 | RESOURCE INLINING / inlineAll | CSS/images/favicons/bg→data: URIs in-page
|
||||
* 22 | 777 | RESOURCE INLINING / inlineAll | CSS/images/favicons/bg→data: URIs in-page (logs errors)
|
||||
* 23 | 887 | SSR RENDER / renderSSR | Full SSR pipeline: acquire→goto→inline→strip
|
||||
* 24 | 932 | SCREENSHOT / renderScreenshotFrame| JPEG screenshot renderer (MJPEG mode)
|
||||
* 25 | 961 | SCREENSHOT HTML / screenshotHTML | Wrap JPEG base64 in refresh-page HTML
|
||||
@@ -1196,7 +1196,8 @@ async function inlineAllResources(page: Page, html: string): Promise<string> {
|
||||
}
|
||||
return collect();
|
||||
});
|
||||
} catch {
|
||||
} catch (err) {
|
||||
console.warn(`[inline] Resource inlining failed: ${err}`);
|
||||
return html;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user