Some ipc messages are not sent when contextIsolation and sandbox are set to true

[pcad]

• Electron version: 1.8.2
• Operating system: MacOS, Windows10

Expected behavior

ipc messages from the main process should be sent to the renderer process when both sandbox and contextIsolation are set.

Actual behavior

ipc messages sent in listeners to webview events are not visible in the renderer process.

How to reproduce

$ git clone https://gist.github.com/4f6c4450cff194838e746ce23e60116d.git
$ cd 4f6c4450cff194838e746ce23e60116d/
$ electron main.js 

The devtools will open and the console will log one message. But the second message on the ipc ping channel is not observed.

Toggling either sandbox or contextIsolation to false allows the ping to travel to the renderer process. In the 1.7.X branch setting both values to true still allows the message to pass through successfully.

[welcome]

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

To help make it easier for us to investigate your issue, please follow the contributing guidelines.

[pcad]

Updated issue to reflect this presents on Windows 10 as well.

[ckerr]

Reproducible in 2.0.x on Ubuntu, as well

[mattbroussard]

I think this might be a dupe of #10868?

[pcad]

It looks like the same issue, but when I try to run this code on 1.7.x line, everything performs as expected. It's only when you move to 1.8 that things stop being sent.

I created a repo that has similar code to the gist above: https://github.com/pcad/electron-ipc-test

Commit 98be5386 uses electron 1.7.15
Commit 70af35f83 uses electron 1.8.2

In 1.7 the message is passed through correctly.
In 1.8 the message is not passed through.

While the commits reference 1.7.15, I also tried 1.7.3 directly to see if something may have happened in some commits in the 1.7.x line.

[SamuelRobotiq]

+1

desktopCapturer.getSources is also not working in preload.js when contextIsolation and sandbox are enabled.

[aegarbutt]

Thanks to @pcad for providing the context around the last version where it worked and first version where it didn't work.