someengineer-nordsec

Current usage of webextension-polyfill pollutes global scope

1

[//]: # (These comments will not show up. You can keep them or delete them, up to you.) #### Describe the bug [//]: # (A clear and concise description of...

Support for manifest version 3

1

official types for MV3 https://www.npmjs.com/package/chrome-types

[Feature/Question]: Exposing _resubscribeTopics

1

Hi, When `resubscribe=true`, mqtt instance tracks resubscribe topics internally (hidden behind private field) It would be helpful, if mqtt instance exposed a method e.g. `getResubscribeTopics()`. Wdyt?

[Feature/Question]: Dynamic reconnectPeriod (read Backoff strategy)

4

Hi, I wanted to get your opinion on this. It seems there is no official approach to have dynamic `reconnectPeriod`. But it is possible to reuse the same concept as...

Happens on all including `0.1.155` ```ts const listener = (windowId: number) => { console.log(windowId); }; chrome.windows.onFocusChanged.addListener(listener); chrome.windows.onFocusChanged.removeListener(listener); chrome.windows.onFocusChanged.hasListener(listener); ``` `addListener`  `removeListener` ![Screenshot 2022-12-20 at...

Remove is-browser.ts detection util in favor of config option

1

**Is your feature request related to a problem? Please describe.** - It seems this detection adds unnecessary complexity - There have been https://github.com/mqttjs/MQTT.js/commits/main/src/lib/is-browser.ts quite a few fixes for it, there...