Skip to content

fix: inherit nodeIntegrationInWorker from the embedder for <webview> and window.open - #52829

Merged
MarshallOfSound merged 1 commit into
41-x-yfrom
trop/41-x-y-bp-fix-inherit-nodeintegrationinworker-from-the-embedder-for-webview-and-window-open-1786835144363
Aug 16, 2026
Merged

MarshallOfSound merged 1 commit into
41-x-yfrom
trop/41-x-y-bp-fix-inherit-nodeintegrationinworker-from-the-embedder-for-webview-and-window-open-1786835144363

Conversation

@trop

@trop trop Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Backport of #52823

See that PR for details.

Notes: <webview> and window.open now inherit nodeIntegrationInWorker from the embedder, consistent with the other Node and sandbox preferences.

…and window.open

The list of security options a <webview> guest always inherits from its
embedder, and the equivalent list child windows inherit from their parent,
omitted nodeIntegrationInWorker. As a result the webpreferences attribute
could set nodeIntegrationInWorker on a guest whose embedder had it disabled,
giving the guest's dedicated workers a Node environment the embedder itself
did not grant (and, with an unsandboxed embedder, dropping the guest out of
the sandbox). Add nodeIntegrationInWorker to both inheritance lists so a
guest can never be less secure than its embedder for this option.

Co-authored-by: Sam Attard <[email protected]>
@trop
trop Bot requested a review from a team as a code owner August 15, 2026 23:05
@trop
trop Bot requested a review from MarshallOfSound August 15, 2026 23:06
@trop trop Bot added 41-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Aug 15, 2026
@MarshallOfSound
MarshallOfSound enabled auto-merge (squash) August 15, 2026 23:06
@MarshallOfSound
MarshallOfSound merged commit 6462a2e into 41-x-y Aug 16, 2026
88 of 90 checks passed
@release-clerk

release-clerk Bot commented Aug 16, 2026

Copy link
Copy Markdown

Release Notes Persisted

&lt;webview&gt; and window.open now inherit nodeIntegrationInWorker from the embedder, consistent with the other Node and sandbox preferences.

@MarshallOfSound
MarshallOfSound deleted the trop/41-x-y-bp-fix-inherit-nodeintegrationinworker-from-the-embedder-for-webview-and-window-open-1786835144363 branch August 16, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

41-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant