From fa221149c2cde5d9f0946f700abaa4ee04c911b7 Mon Sep 17 00:00:00 2001 From: KernelDeimos <7225168+KernelDeimos@users.noreply.github.com> Date: Thu, 9 Oct 2025 16:43:46 -0400 Subject: [PATCH] devex: reduce logs at init Moves more logs to debug level, removing a few that were unnecessary. --- src/backend/src/filesystem/FSNodeContext.js | 2 +- src/backend/src/modules/captcha/services/CaptchaService.js | 4 ++-- src/backend/src/modules/core/LogService.js | 3 ++- src/backend/src/modules/core/ServerHealthService.js | 2 +- src/backend/src/modules/dns/DNSService.js | 2 +- src/backend/src/modules/test-config/TestConfigReadService.js | 2 +- src/backend/src/modules/web/WebServerService.js | 4 ++-- src/backend/src/services/Container.js | 2 +- src/backend/src/services/auth/PermissionService.js | 2 +- src/backend/src/services/drivers/DriverService.js | 2 +- src/gui/webpack/BaseConfig.cjs | 2 +- 11 files changed, 14 insertions(+), 13 deletions(-) diff --git a/src/backend/src/filesystem/FSNodeContext.js b/src/backend/src/filesystem/FSNodeContext.js index 3f5794a99..709714fe2 100644 --- a/src/backend/src/filesystem/FSNodeContext.js +++ b/src/backend/src/filesystem/FSNodeContext.js @@ -323,7 +323,7 @@ module.exports = class FSNodeContext { }, }; - this.log.info('fetching entry: ' + this.selector.describe()); + this.log.debug('fetching entry: ' + this.selector.describe()); const entry = await this.provider.stat({ selector: this.selector, diff --git a/src/backend/src/modules/captcha/services/CaptchaService.js b/src/backend/src/modules/captcha/services/CaptchaService.js index e19b0bdb9..3dac3cd1f 100644 --- a/src/backend/src/modules/captcha/services/CaptchaService.js +++ b/src/backend/src/modules/captcha/services/CaptchaService.js @@ -71,7 +71,7 @@ class CaptchaService extends BaseService { */ async _init() { if (!this.enabled) { - this.log.info('Captcha service is disabled'); + this.log.debug('Captcha service is disabled'); return; } @@ -328,7 +328,7 @@ class CaptchaService extends BaseService { }); this.endpointsRegistered = true; - this.log.info('Captcha service endpoints registered successfully'); + this.log.debug('Captcha service endpoints registered successfully'); // Emit an event that captcha service is ready try { diff --git a/src/backend/src/modules/core/LogService.js b/src/backend/src/modules/core/LogService.js index 1a6f9643f..ddd323f3e 100644 --- a/src/backend/src/modules/core/LogService.js +++ b/src/backend/src/modules/core/LogService.js @@ -524,7 +524,8 @@ class LogService extends BaseService { } this.log = this.create('log-service'); - this.log.system('log service started', { + this.log.system('log service started'); + this.log.debug('log service configuration', { output_lvl: this.output_lvl, log_directory: this.log_directory, }); diff --git a/src/backend/src/modules/core/ServerHealthService.js b/src/backend/src/modules/core/ServerHealthService.js index c506010ea..7846e48aa 100644 --- a/src/backend/src/modules/core/ServerHealthService.js +++ b/src/backend/src/modules/core/ServerHealthService.js @@ -106,7 +106,7 @@ class ServerHealthService extends BaseService { mem_total: meminfo.MemTotal, }; - this.log.info('memory', log_fields); + this.log.debug('memory', log_fields); Object.assign(this.stats_, log_fields); diff --git a/src/backend/src/modules/dns/DNSService.js b/src/backend/src/modules/dns/DNSService.js index 33ae586bf..6c75fc2ff 100644 --- a/src/backend/src/modules/dns/DNSService.js +++ b/src/backend/src/modules/dns/DNSService.js @@ -72,7 +72,7 @@ class DNSService extends BaseService { }); server.on('listening', () => { - this.log.info('Fake DNS server listening', server.addresses()); + this.log.debug('Fake DNS server listening', server.addresses()); if ( this.config.test_server_selftest ) (async () => { await sleep(5000); diff --git a/src/backend/src/modules/test-config/TestConfigReadService.js b/src/backend/src/modules/test-config/TestConfigReadService.js index d0d3c9bfc..67bf3f893 100644 --- a/src/backend/src/modules/test-config/TestConfigReadService.js +++ b/src/backend/src/modules/test-config/TestConfigReadService.js @@ -2,7 +2,7 @@ const BaseService = require("../../services/BaseService"); class TestConfigReadService extends BaseService { async _init () { - this.log.noticeme('test config value (should be abcdefg) is: ' + + this.log.debug('test config value (should be abcdefg) is: ' + this.global_config.testConfigValue, ); } diff --git a/src/backend/src/modules/web/WebServerService.js b/src/backend/src/modules/web/WebServerService.js index 57fa09723..5a06322b1 100644 --- a/src/backend/src/modules/web/WebServerService.js +++ b/src/backend/src/modules/web/WebServerService.js @@ -86,7 +86,7 @@ class WebServerService extends BaseService { return res.sendStatus(200); }); - this.log.noticeme('web server setup done'); + this.log.debug('web server setup done'); } install_post_middlewares_ ({ app }) { @@ -168,7 +168,7 @@ class WebServerService extends BaseService { for ( let i = 0 ; i < ports_to_try.length ; i++ ) { const port = ports_to_try[i]; const is_last_port = i === ports_to_try.length - 1; - if ( auto_port ) this.log.info('trying port: ' + port); + if ( auto_port ) this.log.debug('trying port: ' + port); try { server = http.createServer(this.app).listen(port); server.timeout = 1000 * 60 * 60 * 2; // 2 hours diff --git a/src/backend/src/services/Container.js b/src/backend/src/services/Container.js index 79d23a49a..862abca77 100644 --- a/src/backend/src/services/Container.js +++ b/src/backend/src/services/Container.js @@ -216,7 +216,7 @@ class Container { */ async emit (id, ...args) { if ( this.logger ) { - this.logger.info(`services:event ${id}`, { args }); + this.logger.debug(`services:event ${id}`, { args }); } const promises = []; diff --git a/src/backend/src/services/auth/PermissionService.js b/src/backend/src/services/auth/PermissionService.js index 341cb5354..d4a0e19eb 100644 --- a/src/backend/src/services/auth/PermissionService.js +++ b/src/backend/src/services/auth/PermissionService.js @@ -179,7 +179,7 @@ class PermissionService extends BaseService { } async #scan(actor, permission_options, _reserved, state) { if ( ! state ) { - this.log.info('scan', { + this.log.debug('scan', { actor: actor.uid, permission_options, }); diff --git a/src/backend/src/services/drivers/DriverService.js b/src/backend/src/services/drivers/DriverService.js index d43153948..013058739 100644 --- a/src/backend/src/services/drivers/DriverService.js +++ b/src/backend/src/services/drivers/DriverService.js @@ -467,7 +467,7 @@ class DriverService extends BaseService { effective_policy = effective_policy.policy; - this.log.info('Invoking Driver Call', { + this.log.debug('Invoking Driver Call', { service_name, iface, method, diff --git a/src/gui/webpack/BaseConfig.cjs b/src/gui/webpack/BaseConfig.cjs index f1f7fd1b1..05c38d0f2 100644 --- a/src/gui/webpack/BaseConfig.cjs +++ b/src/gui/webpack/BaseConfig.cjs @@ -34,7 +34,7 @@ module.exports = async (options = {}) => { const entries = []; for ( const extensionsDir of extension_directories ) { - console.log(`Reading extensions from ${extensionsDir}`); + // console.log(`Reading extensions from ${extensionsDir}`); // Read and process extension entries from the extensions directory const readdir_entries = fs.readdirSync(extensionsDir, { withFileTypes: true }) for ( const entry of readdir_entries ) {