Merge pull request #9 from joergbasedow/master

Allow overriding of system name from config

Author: hedii

readme.md

@@ -65,6 +65,11 @@ return [
             // configuration value
             'name' => 'my-custom-name',
 
+            // This optional option determines the system name sent with the
+            // message in the 'source' field. When Forgotten or set to null,
+            // the current hostname is used.
+            'system_name' => null,
+
             // This optional option determines the host that will receive the
             // gelf log messages. Default is 127.0.0.1
             'host' => '127.0.0.1',

src/GelfLoggerFactory.php

@@ -63,7 +63,7 @@ public function __invoke(array $config): Logger
 
         $handler = new GelfHandler(new Publisher($transport), $this->level($config));
 
-        $handler->setFormatter(new GelfMessageFormatter(null, null, null));
+        $handler->setFormatter(new GelfMessageFormatter($config['system_name'] ?? null, null, null));
 
         foreach ($this->parseProcessors($config) as $processor) {
             $handler->pushProcessor(new $processor);

tests/GelfLoggerTest.php

@@ -58,4 +58,32 @@ public function it_should_not_have_any_processor_if_the_config_does_not_have_pro
 
         $handler->popProcessor();
     }
+
+    /** @test */
+    public function it_should_set_system_name_to_current_hostname_if_system_name_is_null(): void
+    {
+        $this->app['config']->set('logging.channels.gelf', [
+            'system_name' => null,
+            'driver' => 'custom',
+            'via' => GelfLoggerFactory::class
+        ]);
+
+        $logger = Log::channel('gelf');
+
+        $this->assertAttributeEquals(gethostname(), 'systemName', $logger->getHandlers()[0]->getFormatter());
+    }
+
+    /** @test */
+    public function it_should_set_system_name_to_custom_value_if_system_name_config_is_provided(): void
+    {
+        $this->app['config']->set('logging.channels.gelf', [
+            'system_name' => 'my-system-name',
+            'driver' => 'custom',
+            'via' => GelfLoggerFactory::class
+        ]);
+
+        $logger = Log::channel('gelf');
+
+        $this->assertAttributeEquals('my-system-name', 'systemName', $logger->getHandlers()[0]->getFormatter());
+    }
 }