Do not throw an exception when the samlAuthenticationLogger is used before forAuthentication() is called on it

This obscures the original log message

Author: pmeulen

src/Monolog/SamlAuthenticationLogger.php

@@ -96,12 +96,10 @@ public function log($level, $message, array $context = []): void
      */
     private function modifyContext(array $context): array
     {
-        if (!$this->sari) {
-            throw new RuntimeException('Authentication logging context is unknown');
+        if ($this->sari) {
+            $context['sari'] = $this->sari;
         }
 
-        $context['sari'] = $this->sari;
-
         return $context;
     }
 }

src/Tests/Unit/Monolog/SamlAuthenticationLoggerTest.php

@@ -47,10 +47,12 @@ public function it_returns_a_logger_for_an_authentication(): void
     /**
      * @test
      */
-    public function it_errors_when_no_authentication(): void
+    public function it_does_not_throw_when_no_authentication(): void
     {
-        self::expectException(RuntimeException::class);
-        $logger = new SamlAuthenticationLogger(m::mock(LoggerInterface::class));
+        $innerLogger = m::mock(LoggerInterface::class);
+        $innerLogger->shouldReceive('emergency')->with('message2', [])->once();
+
+        $logger = new SamlAuthenticationLogger($innerLogger);
         $logger->emergency('message2');
     }
 }