New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: do not register the node ESM loader in renderer processes #24301
Conversation
Is it possible to add test for this change ? |
@deepak1556 and @MarshallOfSound i think yes - we added the sandbox change to ARM to account for this here, so we should be able to remove it now. If it fails, we know this didn't have the intended effect. |
Failing vanilla node tests are related. |
oh boy missed those - i think we'll need to ensure it's enabled in run as node mode @MarshallOfSound |
@MarshallOfSound can you rebase this against the accepted upstream solution: nodejs/node#34060 |
No Release Notes |
Only one ESM loader can be registered per isolate, in renderer processes this should be blink. This patches node so that it won't register its handler (overriding blinks) in non-browser processes.
Notes: no-notes