jens/flipper
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Handle onMessage errors when client is not connected

Browse Source 
Reviewed By: lblasa

Differential Revision: D47397641

fbshipit-source-id: ac9c057050a3bac0c5149f5ce2998657fededb61
This commit is contained in:
Andrey Goncharov
2023-07-14 03:31:20 -07:00
committed by Facebook GitHub Bot
parent 257003af42
commit 0345d7d533
1 changed files with 31 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

54
desktop/flipper-frontend-core/src/AbstractClient.tsx
View File

@@ -218,30 +218,38 @@ export default abstract class AbstractClient extends EventEmitter {
// get the plugins, and update the UI
async refreshPlugins() {
const oldBackgroundPlugins = this.backgroundPlugins;
await this.loadPlugins('refresh');
await Promise.all(
[...this.plugins].map(async (pluginId) =>
this.startPluginIfNeeded(await this.getPlugin(pluginId)),
),
);
const newBackgroundPlugins = await this.getBackgroundPlugins();
this.backgroundPlugins = new Set(newBackgroundPlugins);
// diff the background plugin list, disconnect old, connect new ones
oldBackgroundPlugins.forEach((plugin) => {
if (!this.backgroundPlugins.has(plugin)) {
this.deinitPlugin(plugin);
try {
const oldBackgroundPlugins = this.backgroundPlugins;
await this.loadPlugins('refresh');
await Promise.all(
[...this.plugins].map(async (pluginId) =>
this.startPluginIfNeeded(await this.getPlugin(pluginId)),
),
);
const newBackgroundPlugins = await this.getBackgroundPlugins();
this.backgroundPlugins = new Set(newBackgroundPlugins);
// diff the background plugin list, disconnect old, connect new ones
oldBackgroundPlugins.forEach((plugin) => {
if (!this.backgroundPlugins.has(plugin)) {
this.deinitPlugin(plugin);
}
});
newBackgroundPlugins.forEach((plugin) => {
if (
!oldBackgroundPlugins.has(plugin) &&
this.shouldConnectAsBackgroundPlugin(plugin)
) {
this.initPlugin(plugin);
}
});
this.emit('plugins-change');
} catch (e) {
if (this.connected) {
throw e;
} else {
console.warn(e);
}
});
newBackgroundPlugins.forEach((plugin) => {
if (
!oldBackgroundPlugins.has(plugin) &&
this.shouldConnectAsBackgroundPlugin(plugin)
) {
this.initPlugin(plugin);
}
});
this.emit('plugins-change');
}
}
onMessage(msg: string) {