cleanup

composer.json

@@ -16,11 +16,7 @@
     }
   ],
   "require": {
-    "monolog/monolog": "^3.0",
-    "nyholm/dsn": "^2.0",
-    "nyholm/psr7": "^1.5",
     "open-telemetry/sdk": "^1.1",
-    "psr/http-client": "^1.0",
     "symfony/dependency-injection": "*"
   },
   "require-dev": {
@@ -40,6 +36,10 @@
     "symfony/twig-bundle": "*",
     "worldia/coding-standards": "^1.0"
   },
+  "suggest": {
+    "open-telemetry/exporter-otlp": "To export telemetry data to an OTLP compatible ingester (Jaeger, Tempo, Loki, etc.)",
+    "symfony/monolog-bundle": "To enable open-telemetry logging"
+  },
   "autoload": {
     "psr-4": {
       "Instrumentation\\": "src/"

src/DependencyInjection/Extension.php

@@ -13,6 +13,7 @@
 use Instrumentation\Tracing\Bridge\TraceUrlGeneratorInterface;
 use Instrumentation\Tracing\Doctrine\Instrumentation\DBAL\Middleware as InstrumentationMiddleware;
 use Instrumentation\Tracing\Doctrine\Propagation\DBAL\Middleware as PropagationMiddleware;
+use Symfony\Bundle\MonologBundle\MonologBundle;
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -188,6 +189,10 @@ protected function loadTracing(array $config, ContainerBuilder $container): void
      */
     protected function loadLogging(array $config, ContainerBuilder $container): void
     {
+        if (!class_exists(MonologBundle::class)) {
+            throw new \InvalidArgumentException('Logging requires "symfony/monolog-bundle". Did you forget to add it?');
+        }
+
         $loader = $this->getLoader('logging', $container);
 
         $loader->load('logging.php');

src/Semantics/Attribute/ClientRequestAttributeProvider.php

@@ -9,7 +9,6 @@
 
 namespace Instrumentation\Semantics\Attribute;
 
-use Instrumentation\Tracing\HttpClient\HttpSensitiveDataHelper;
 use OpenTelemetry\SemConv\TraceAttributes;
 
 class ClientRequestAttributeProvider implements ClientRequestAttributeProviderInterface
@@ -25,7 +24,7 @@ public function getAttributes(string $method, string $url, array $headers = []):
     {
         $attributes = [
             TraceAttributes::HTTP_REQUEST_METHOD => strtoupper($method),
-            TraceAttributes::URL_FULL => HttpSensitiveDataHelper::filterUrl($url),
+            TraceAttributes::URL_FULL => preg_replace('|(^https?://(?:.*):)(?:.*)(@.*)|', '$1<redacted>$2', $url),
         ];
 
         foreach ($this->capturedHeaders as $header) {