fix(logs): use correct option name

phpstan-baseline.neon

@@ -86,7 +86,7 @@ parameters:
 			path: src/DependencyInjection/SentryExtension.php
 
 		-
-			message: "#^Cannot access offset 'before_send_logs' on mixed\\.$#"
+			message: "#^Cannot access offset 'before_send_log' on mixed\\.$#"
 			count: 1
 			path: src/DependencyInjection/SentryExtension.php
 

src/DependencyInjection/Configuration.php

@@ -113,7 +113,7 @@ public function getConfigTreeBuilder(): TreeBuilder
                         ->scalarNode('before_send_transaction')->end()
                         ->scalarNode('before_send_check_in')->end()
                         ->scalarNode('before_send_metrics')->end()
-                        ->scalarNode('before_send_logs')->end()
+                        ->scalarNode('before_send_log')->end()
                         ->variableNode('trace_propagation_targets')->end()
                         ->arrayNode('tags')
                             ->useAttributeAsKey('name')

src/DependencyInjection/SentryExtension.php

@@ -124,8 +124,8 @@ private function registerConfiguration(ContainerBuilder $container, array $confi
             $options['before_send_metrics'] = new Reference($options['before_send_metrics']);
         }
 
-        if (isset($options['before_send_logs'])) {
-            $options['before_send_logs'] = new Reference($options['before_send_logs']);
+        if (isset($options['before_send_log'])) {
+            $options['before_send_log'] = new Reference($options['before_send_log']);
         }
 
         if (isset($options['before_breadcrumb'])) {

src/Resources/config/schema/sentry-1.0.xsd

@@ -55,7 +55,7 @@
         <xsd:attribute name="before-send-transaction" type="xsd:string" />
         <xsd:attribute name="before-send-check-in" type="xsd:string" />
         <xsd:attribute name="before-send-metrics" type="xsd:string" />
-        <xsd:attribute name="before-send-logs" type="xsd:string" />
+        <xsd:attribute name="before-send-log" type="xsd:string" />
         <xsd:attribute name="error-types" type="xsd:string" />
         <xsd:attribute name="max-breadcrumbs" type="xsd:integer" />
         <xsd:attribute name="before-breadcrumb" type="xsd:string" />

tests/DependencyInjection/Fixtures/php/full.php

@@ -33,7 +33,7 @@
         'before_send_transaction' => 'App\\Sentry\\BeforeSendTransactionCallback',
         'before_send_check_in' => 'App\\Sentry\\BeforeSendCheckInCallback',
         'before_send_metrics' => 'App\\Sentry\\BeforeSendMetricsCallback',
-        'before_send_logs' => 'App\\Sentry\\BeforeSendLogsCallback',
+        'before_send_log' => 'App\\Sentry\\BeforeSendLogsCallback',
         'trace_propagation_targets' => ['website.invalid'],
         'tags' => [
             'context' => 'development',

tests/DependencyInjection/Fixtures/xml/full.xml

@@ -30,7 +30,7 @@
                         before-send-transaction="App\Sentry\BeforeSendTransactionCallback"
                         before-send-check-in="App\Sentry\BeforeSendCheckInCallback"
                         before-send-metrics="App\Sentry\BeforeSendMetricsCallback"
-                        before-send-logs="App\Sentry\BeforeSendLogsCallback"
+                        before-send-log="App\Sentry\BeforeSendLogsCallback"
                         error-types="E_ALL"
                         max-breadcrumbs="1"
                         before-breadcrumb="App\Sentry\BeforeBreadcrumbCallback"

tests/DependencyInjection/Fixtures/yml/full.yml

@@ -30,7 +30,7 @@ sentry:
         before_send_transaction: App\Sentry\BeforeSendTransactionCallback
         before_send_check_in: App\Sentry\BeforeSendCheckInCallback
         before_send_metrics: App\Sentry\BeforeSendMetricsCallback
-        before_send_logs: App\Sentry\BeforeSendLogsCallback
+        before_send_log: App\Sentry\BeforeSendLogsCallback
         trace_propagation_targets:
           - 'website.invalid'
         tags:

tests/DependencyInjection/SentryExtensionTest.php

@@ -229,7 +229,7 @@ public function testClientIsCreatedFromOptions(): void
             'before_send_transaction' => new Reference('App\\Sentry\\BeforeSendTransactionCallback'),
             'before_send_check_in' => new Reference('App\\Sentry\\BeforeSendCheckInCallback'),
             'before_send_metrics' => new Reference('App\\Sentry\\BeforeSendMetricsCallback'),
-            'before_send_logs' => new Reference('App\\Sentry\\BeforeSendLogsCallback'),
+            'before_send_log' => new Reference('App\\Sentry\\BeforeSendLogsCallback'),
             'trace_propagation_targets' => ['website.invalid'],
             'tags' => [
                 'context' => 'development',

tests/End2End/App/Callback/BeforeSendLogCallback.php

@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sentry\SentryBundle\Tests\End2End\App\Callback;
+
+use Sentry\Logs\Log;
+
+class BeforeSendLogCallback
+{
+    public function getCallback(): callable
+    {
+        return function (Log $log): ?Log {
+            if ('before_send_log' === $log->getBody()) {
+                return null;
+            }
+
+            return $log;
+        };
+    }
+}

tests/End2End/App/Controller/LoggingController.php

@@ -43,4 +43,13 @@ public function loggingWithError()
         $this->logger->error('About to crash');
         throw new \RuntimeException('Crash');
     }
+
+    public function beforeSendLog()
+    {
+        $this->logger->warning('warn 1');
+        $this->logger->error('before_send_log');
+        $this->logger->warning('warn 2');
+
+        return new Response();
+    }
 }

tests/End2End/App/logging.yml

@@ -1,6 +1,7 @@
 sentry:
   options:
     enable_logs: true
+    before_send_log: "sentry.callback.before_send_log"
 
 monolog:
   handlers:
@@ -13,6 +14,13 @@ services:
     arguments:
       - !php/const Monolog\Logger::WARNING
 
+  Sentry\SentryBundle\Tests\End2End\App\Callback\BeforeSendLogCallback:
+    autowire: true
+
+  sentry.callback.before_send_log:
+    class: 'Sentry\SentryBundle\Tests\End2End\App\Callback\BeforeSendLogCallback'
+    factory: ['@Sentry\SentryBundle\Tests\End2End\App\Callback\BeforeSendLogCallback', 'getCallback']
+
   Sentry\SentryBundle\Tests\End2End\App\Controller\LoggingController:
     autowire: true
     tags:

tests/End2End/App/routing.yml

@@ -50,6 +50,10 @@ logging_With_Error:
   path: /logging-with-error
   defaults: { _controller: 'Sentry\SentryBundle\Tests\End2End\App\Controller\LoggingController::loggingWithError' }
 
+logging_before_send_logs:
+  path: /before-send-log
+  defaults: { _controller: 'Sentry\SentryBundle\Tests\End2End\App\Controller\LoggingController::beforeSendLog' }
+
 buffer_flush:
   path: /buffer-flush
   defaults: { _controller: 'Sentry\SentryBundle\Tests\End2End\App\Controller\BufferFlushController::testBufferFlush' }

tests/End2End/LoggingEnd2EndTest.php

@@ -57,6 +57,23 @@ public function testExceptionWithLogs(): void
         $this->assertCount(2, $logsEvent->getLogs());
     }
 
+    public function testBeforeSendLogCallback(): void
+    {
+        $client = static::createClient(['debug' => false]);
+
+        $client->request('GET', '/before-send-log');
+        $this->assertSame(200, $client->getResponse()->getStatusCode());
+
+        $this->assertCount(1, StubTransport::$events);
+        $event = StubTransport::$events[0];
+        $logs = $this->filterFrameworkLogs($event->getLogs());
+
+        // Make sure we just have two warn logs and no error log (since it's filtered by the callback)
+        $this->assertCount(2, $logs);
+        $this->assertEquals('warn 1', $logs[0]->getBody());
+        $this->assertEquals('warn 2', $logs[1]->getBody());
+    }
+
     /**
      * Removes framework logs so that the tests can focus on our expected logs.
      *