fix: fall back to low cardinality

Author: cedricziel

src/Instrumentation/Laravel/src/Hooks/Illuminate/Contracts/Http/Kernel.php

@@ -44,7 +44,7 @@ protected function hookHandle(): bool
                 /** @psalm-suppress ArgumentTypeCoercion */
                 $builder = $this->instrumentation
                     ->tracer()
-                    ->spanBuilder(sprintf('%s', $request?->method() ?? 'unknown'))
+                    ->spanBuilder(sprintf('HTTP %s', $request?->method() ?? 'unknown'))
                     ->setSpanKind(SpanKind::KIND_SERVER)
                     ->setAttribute(TraceAttributes::CODE_FUNCTION_NAME, $function)
                     ->setAttribute(TraceAttributes::CODE_NAMESPACE, $class)
@@ -87,7 +87,12 @@ protected function hookHandle(): bool
                 $route = $request?->route();
 
                 if ($request && $route instanceof Route) {
-                    $span->updateName("{$request->method()} /" . ltrim($route->uri, '/'));
+                    if (method_exists($route, 'getName') && $route->getName() && strpos($route->getName(), 'generated::') !== 0) {
+                        $span->updateName("{$request->method()} " . $route->getName());
+                        $span->setAttribute('laravel.route.name', $route->getName());
+                    } else {
+                        $span->updateName("HTTP {$request->method()}");
+                    }
                     $span->setAttribute(TraceAttributes::HTTP_ROUTE, $route->uri);
                 }
 

src/Instrumentation/Laravel/src/Hooks/Illuminate/Routing/Router.php

@@ -75,13 +75,20 @@ protected function hookPrepareResponse(): bool
                 if (method_exists($route, 'getName') && $route->getName() && strpos($route->getName(), 'generated::') !== 0) {
                     $span->updateName("{$request->method()} " . $route->getName());
                     $span->setAttribute('laravel.route.name', $route->getName());
-                } elseif (method_exists($route, 'uri')) {
+                }
+
+                // Always set the HTTP route attribute from the URI pattern
+                if (method_exists($route, 'uri')) {
                     $path = $route->uri();
-                    $span->updateName("{$request->method()} /" . ltrim($path, '/'));
+                    if (!$route->getName() || strpos($route->getName(), 'generated::') === 0) {
+                        $span->updateName("HTTP {$request->method()}");
+                    }
                     $span->setAttribute(TraceAttributes::HTTP_ROUTE, $path);
                 } elseif (method_exists($route, 'getPath')) {
                     $path = $route->getPath();
-                    $span->updateName("{$request->method()} /" . ltrim($path, '/'));
+                    if (!$route->getName() || strpos($route->getName(), 'generated::') === 0) {
+                        $span->updateName("HTTP{$request->method()}");
+                    }
                     $span->setAttribute(TraceAttributes::HTTP_ROUTE, $path);
                 }