Use Sentry default integrations

This activates the default integrations shipped with Sentry again.

This adds information about installed Composer packages, the request,
frame context and environment.

Through this, some "custom" tags can be dropped, as the information is
included by the default integrations.

Since Sentry registers an error handler, errors not handled by the
exception handler will also be recorded in Sentry.

Author: kdambekalns

Classes/SentryClient.php

@@ -17,8 +17,6 @@
 use Flownative\Sentry\Context\UserContextServiceInterface;
 use Flownative\Sentry\Context\WithExtraDataInterface;
 use Flownative\Sentry\Log\CaptureResult;
-use GuzzleHttp\Psr7\ServerRequest;
-use Jenssegers\Agent\Agent;
 use Neos\Flow\Annotations as Flow;
 use Neos\Flow\Core\Bootstrap;
 use Neos\Flow\Error\WithReferenceCodeInterface;
@@ -31,7 +29,6 @@
 use Neos\Flow\Utility\Environment;
 use Neos\Utility\Arrays;
 use Psr\Log\LoggerInterface;
-use Psr\Log\LogLevel;
 use Sentry\Event;
 use Sentry\EventHint;
 use Sentry\EventId;
@@ -93,6 +90,10 @@ public function injectSettings(array $settings): void
 
     public function initializeObject(): void
     {
+        if (empty($this->dsn)) {
+            return;
+        }
+
         $representationSerializer = new RepresentationSerializer(
             new Options([])
         );
@@ -102,10 +103,6 @@ public function initializeObject(): void
             $representationSerializer
         );
 
-        if (empty($this->dsn)) {
-            return;
-        }
-
         \Sentry\init([
             'dsn' => $this->dsn,
             'environment' => $this->environment,
@@ -118,8 +115,7 @@ public function initializeObject(): void
                 FLOW_PATH_ROOT . '/Packages/Framework/Neos.Flow/Classes/Log/',
                 FLOW_PATH_ROOT . '/Packages/Libraries/neos/flow-log/'
             ],
-            'default_integrations' => false,
-            'attach_stacktrace' => true
+            'attach_stacktrace' => true,
         ]);
 
         $client = SentrySdk::getCurrentHub()->getClient();
@@ -136,38 +132,18 @@ private function setTags(): void
             try {
                 $flowPackage = $this->packageManager->getPackage('Neos.Flow');
                 $flowVersion = $flowPackage->getInstalledVersion();
-            } catch (UnknownPackageException $e) {
+            } catch (UnknownPackageException) {
             }
         }
         if (empty($flowVersion)) {
             $flowVersion = FLOW_VERSION_BRANCH;
         }
 
-        $currentSession = null;
-        if ($this->sessionManager) {
-            $currentSession = $this->sessionManager->getCurrentSession();
-        }
+        $currentSession = $this->sessionManager?->getCurrentSession();
 
         SentrySdk::getCurrentHub()->configureScope(static function (Scope $scope) use ($flowVersion, $currentSession): void {
             $scope->setTag('flow_version', $flowVersion);
             $scope->setTag('flow_context', (string)Bootstrap::$staticObjectManager->get(Environment::class)->getContext());
-            $scope->setTag('php_version', PHP_VERSION);
-
-            if (PHP_SAPI !== 'cli') {
-                $scope->setTag('uri',
-                    (string)ServerRequest::fromGlobals()->getUri());
-
-                $agent = new Agent();
-                $scope->setContext('client_os', [
-                    'name' => $agent->platform(),
-                    'version' => $agent->version($agent->platform())
-                ]);
-
-                $scope->setContext('client_browser', [
-                    'name' => $agent->browser(),
-                    'version' => $agent->version($agent->browser())
-                ]);
-            }
 
             if ($currentSession instanceof Session && $currentSession->isStarted()) {
                 $scope->setTag('flow_session_sha1', sha1($currentSession->getId()));
@@ -226,7 +202,7 @@ public function captureThrowable(Throwable $throwable, array $extraData = [], ar
             $message = 'ignored';
         }
         return new CaptureResult(
-    true,
+            true,
             $message,
             (string)$sentryEventId
         );
@@ -326,7 +302,7 @@ private function prepareStacktrace(\Throwable $throwable = null): Stacktrace
                 $frame->getRawFunctionName(),
                 $frame->getAbsoluteFilePath(),
                 $frame->getVars(),
-                strpos($classPathAndFilename, 'Packages/Framework/') === false
+                !str_contains($classPathAndFilename, 'Packages/Framework/')
             );
         }
         return new Stacktrace($frames);

composer.json

@@ -17,8 +17,7 @@
         "ext-json": "*",
         "php": "^8.1",
         "neos/flow": "^8.0 || ^9.0 || @dev",
-        "sentry/sentry": "^4.0",
-        "jenssegers/agent": "^2.6"
+        "sentry/sentry": "^4.0"
     },
     "autoload": {
         "psr-4": {