[Bug]: WeChat mini globalThis is not defined in browser bundle

[Diven-xinhai]

MQTTjs Version

5.3.6

Broker

mqtt

Environment

NodeJS

Description

The latest version 5.3.6 reported an error in the wechat mini program, the error message is as follows:
TypeError: Cannot read property 'AbortController' of undefined

Minimal Reproduction

Debug logs

.

[Diven-xinhai]

[robertsLando]

Could you try to add this to the mqttjs bundle?

  globalThis = globalThis || {};

[Diven-xinhai]

Could you try to add this to the mqttjs bundle?

  globalThis = globalThis || {};

Sorry, I don't quite understand what you mean. Could you please specify where or in which file I should add?

[robertsLando]

@Diven-xinhai node_modules/mqtt/dist/mqtt.esm.js

Alternatively you could also try to add that in a script tag in your index.html file

[Diven-xinhai]

@Diven-xinhai node_modules/mqtt/dist/mqtt.esm.js

Alternatively you could also try to add that in a script tag in your index.html file

I tried to add globalThis = globalThis || {} in index.html or mqtt.esm.js; But it didn't work. It's the same error

[robertsLando]

The problem is that for some reason when you are running this in wechat mini globalThis is not defined (it should be in all browsers BTW) so you should try fixing it within your bundle someway because I don't have a way to test this out

[timzaak]

MQTTjs Version
5.3.6

TypeError: Cannot set property 'AbortSignal' of undefined

Another issue about this is NervJS/taro#15580

[robertsLando]

@timzaak I don't use wechat and I also have no way to test this so please feel free to open a PR to fix this issue.

[timzaak]

@timzaak I don't use wechat and I also have no way to test this so please feel free to open a PR to fix this issue.

@robertsLando I solved this by import 'abortcontroller-polyfill/dist/abortcontroller-polyfill-only' before import mqtt

but worker-timers uses new Blob(), Wechat miniprogram do not support Blob, it should be replaced by ArrayBuffer.

https://github.com/chrisguttandin/worker-timers/blob/8f73b936a20073a809b180d4557c657f26e4e7fa/src/factories/load-or-return-broker.ts#L9

[robertsLando]

@timzaak You can disable using worker timer with timerVariant option

[timzaak]

@timzaak You can disable using worker timer with timerVariant option

@robertsLando That works. Thanks