Electron force mqtts instead of wss

[axi92]

We used mqtts before version 5 of the lib and it worked fine. Now we want to upgrade but it always falls back to ws and tries to open a connection with that.
I tried to force it to use mqtt, the only part where I had success was by modifying the source isWebWorkerEnv to return false so it would connect via mqtt and that worked.

So my question is is there a normal way without forking the lib?

I have to add the electron webPreferences are:

        webPreferences: {
            sandbox: false,
            nodeIntegration: true,
            contextIsolation: false,
            zoomFactor: touchZoomFactor,
        },

[robertsLando]

In order to fix this we should identify electron env and if so return false in isBrowser. Would you like to submit a PR?

[robertsLando]

But here it imports an object

This is ESM, by default it exports the default export, with require it would be equal to:

 const IS_BROWSER = require('../is-browser').default 

Anyway that's the compiled code of esbuilf, ignore it, that's something you should not work on, you should work on typescript code.

[axi92]

I am having a hard time to execute the tests even on the main branch they fail at different stages.
Is this a known issue?
After several tries I was able to get a green result.

Any thoughts?
main...axi92:MQTT.js:feature/electron-context

I was able to test it in the render and in the main process from electron. But I would be happy if somebody else could confirm that.

[robertsLando]

Open the pr please. About tests them are flacky but it's known

[axi92]

#1856 (comment)