Skip to content

Commit 7010b37

Browse files
committed
Small refactoring
1 parent 537f1f1 commit 7010b37

File tree

1 file changed

+10
-15
lines changed

1 file changed

+10
-15
lines changed

src/DDTrace/Integrations/Symfony/SymfonyIntegration.php

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -388,23 +388,12 @@ function () use ($integration) {
388388
}
389389
);
390390

391-
$handle_route = function($request, $rootSpan) use ($integration) {
392-
$route_name = $request->get('_route');
393-
if ($route_name === null) {
394-
return;
395-
}
396-
if (dd_trace_env_config("DD_HTTP_SERVER_ROUTE_BASED_NAMING")) {
397-
$rootSpan->resource = $route_name;
398-
}
399-
$rootSpan->meta['symfony.route.name'] = $route_name;
400-
401-
// the rest is for determining http.route
391+
$handle_http_route = function($route_name, $request, $rootSpan) use ($integration) {
402392
if ($integration->kernel === null) {
403393
return;
404394
}
405395
/** @var ContainerInterface $container */
406396
$container = $integration->kernel->getContainer();
407-
$cache = null;
408397
try {
409398
$cache = $container->get('cache.app');
410399
} catch (\Exception $e) {
@@ -437,13 +426,12 @@ function () use ($integration) {
437426
if (isset($route)) {
438427
$rootSpan->meta[Tag::HTTP_ROUTE] = $route->getPath();
439428
}
440-
441429
};
442430

443431
\DDTrace\trace_method(
444432
'Symfony\Component\HttpKernel\HttpKernel',
445433
'handle',
446-
function (SpanData $span, $args, $response) use ($integration, $handle_route) {
434+
function (SpanData $span, $args, $response) use ($integration, $handle_http_route) {
447435
/** @var Request $request */
448436
list($request) = $args;
449437

@@ -465,7 +453,14 @@ function (SpanData $span, $args, $response) use ($integration, $handle_route) {
465453
$rootSpan->meta[Tag::HTTP_STATUS_CODE] = $response->getStatusCode();
466454
}
467455

468-
$handle_route($request, $rootSpan);
456+
$route_name = $request->get('_route');
457+
if ($route_name !== null) {
458+
if (dd_trace_env_config("DD_HTTP_SERVER_ROUTE_BASED_NAMING")) {
459+
$rootSpan->resource = $route_name;
460+
}
461+
$rootSpan->meta['symfony.route.name'] = $route_name;
462+
$handle_http_route($route_name, $request, $rootSpan);
463+
}
469464

470465
$parameters = $request->get('_route_params');
471466
if (!empty($parameters) &&

0 commit comments

Comments
 (0)