By default, only override the config if the override tenant property is set (otherwise, just skip the override and keep the default config value)

Author: lukinovec

src/Bootstrappers/LogTenancyBootstrapper.php

@@ -100,7 +100,9 @@ protected function overrideChannelConfig(string $channel, array|Closure $overrid
         if (is_array($override)) {
             // Map tenant properties to channel config keys
             foreach ($override as $configKey => $tenantProperty) {
-                $this->config->set("logging.channels.{$channel}.{$configKey}", $tenant->$tenantProperty);
+                if ($tenant->$tenantProperty) {
+                    $this->config->set("logging.channels.{$channel}.{$configKey}", $tenant->$tenantProperty);
+                }
             }
         } elseif ($override instanceof Closure) {
             // Execute custom configuration closure

tests/Bootstrappers/LogTenancyBootstrapperTest.php

@@ -150,6 +150,20 @@
     expect(config('logging.channels.slack.username'))->toBe('Default');
 });
 
+test('channel config keys remains unchanged if the specified tenant override property is not set', function() {
+    config(['logging.default' => 'slack']);
+    config(['logging.channels.slack.username' => 'Default username']);
+
+    LogTenancyBootstrapper::$channelOverrides = [
+        'slack' => ['username' => 'nonExistentProperty'], // $tenant->nonExistentProperty
+    ];
+
+    tenancy()->initialize(Tenant::create());
+
+    // The username should remain unchanged since the tenant property is not set
+    expect(config('logging.channels.slack.username'))->toBe('Default username');
+});
+
 test('channel overrides take precedence over the default storage path channel updating logic', function () {
     config(['logging.default' => 'single']);