yuuza/outline
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: Duplicate plugin registration logging in debug

Browse Source
This commit is contained in:
Tom Moor
2024-05-03 08:15:56 -04:00
parent adc4f2b544
commit 9204a8ab30
2 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
server/index.ts
View File

@@ -59,7 +59,7 @@ async function master() {
} }
// This function will only be called in each forked process // This function will only be called in each forked process
async function start(id: number, disconnect: () => void) { async function start(_id: number, disconnect: () => void) {
// Ensure plugins are loaded // Ensure plugins are loaded
PluginManager.loadPlugins(); PluginManager.loadPlugins();

16
server/utils/PluginManager.ts
View File

@@ -81,12 +81,16 @@ export class PluginManager {
.get(plugin.type)! .get(plugin.type)!
.push({ ...plugin, priority: plugin.priority ?? PluginPriority.Normal }); .push({ ...plugin, priority: plugin.priority ?? PluginPriority.Normal });
Logger.debug( // Do not log plugin registration in forked worker processes, one log from the master process
"plugins", // is enough. This can be detected by the presence of `process.send`.
`Plugin(type=${plugin.type}) registered ${ if (process.send === undefined) {
"name" in plugin.value ? plugin.value.name : "" Logger.debug(
} ${plugin.description ? `(${plugin.description})` : ""}` "plugins",
); `Plugin(type=${plugin.type}) registered ${
"name" in plugin.value ? plugin.value.name : ""
} ${plugin.description ? `(${plugin.description})` : ""}`
);
}
} }
/** /**