Skip to content

Commit fae950a

Browse files
committed
Merge branch '2.0.x' into 1.0.x
# Conflicts: # composer.json
2 parents 9d8b510 + 3f7bfaa commit fae950a

File tree

5 files changed

+44
-2
lines changed

5 files changed

+44
-2
lines changed

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"symfony/console": "~2 || ~3",
1313
"symfony/http-kernel": "~2 || ~3",
1414
"symfony/dependency-injection": "~2 || ~3",
15-
"code-tool/jaeger-client-php": "~1"
15+
"code-tool/jaeger-client-php": "~1.1"
1616
},
1717
"require-dev": {
1818
"php-ds/php-ds": "@stable",

src/Bridge/AppStartSpanListener.php

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,11 @@
11
<?php
22
namespace Jaeger\Symfony\Bridge;
33

4+
use Jaeger\Symfony\Tag\SymfonyComponentTag;
5+
use Jaeger\Symfony\Tag\SymfonyVersionTag;
46
use Jaeger\Tag\DoubleTag;
57
use Jaeger\Tag\LongTag;
8+
use Jaeger\Tag\SpanKindServerTag;
69
use Jaeger\Tag\StringTag;
710
use Jaeger\Tracer\TracerInterface;
811
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
@@ -34,7 +37,10 @@ public function onRequest(GetResponseEvent $event)
3437
$value = $request->server->get('REQUEST_TIME_FLOAT', microtime(true));
3538
$startTime = (int)($value * 1000000);
3639
$this->tracer->finish(
37-
$this->tracer->start('app.start')
40+
$this->tracer->start('symfony.start')
41+
->addTag(new SpanKindServerTag())
42+
->addTag(new SymfonyComponentTag())
43+
->addTag(new SymfonyVersionTag())
3844
->addTag(new StringTag('time.source', $source))
3945
->addTag(new DoubleTag('time.value', $value))
4046
->addTag(new LongTag('time.micro', $startTime))

src/Bridge/RequestSpanListener.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,11 @@
44
use Jaeger\Http\HttpCodeTag;
55
use Jaeger\Http\HttpMethodTag;
66
use Jaeger\Http\HttpUriTag;
7+
use Jaeger\Symfony\Tag\SymfonyComponentTag;
8+
use Jaeger\Symfony\Tag\SymfonyVersionTag;
79
use Jaeger\Tag\DoubleTag;
810
use Jaeger\Tag\LongTag;
11+
use Jaeger\Tag\SpanKindServerTag;
912
use Jaeger\Tag\StringTag;
1013
use Jaeger\Tracer\TracerInterface;
1114
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
@@ -67,6 +70,9 @@ public function onRequest(GetResponseEvent $event)
6770
[
6871
new HttpMethodTag($request->getMethod()),
6972
new HttpUriTag($request->getRequestUri()),
73+
new SpanKindServerTag(),
74+
new SymfonyComponentTag(),
75+
new SymfonyVersionTag()
7076
]
7177
);
7278

@@ -80,6 +86,7 @@ public function onRequest(GetResponseEvent $event)
8086
->addTag(new LongTag('time.micro', $startTime))
8187
->start($startTime);
8288
}
89+
8390
$this->spans->push($requestSpan);
8491

8592
return $this;

src/Tag/SymfonyComponentTag.php

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
<?php
2+
declare(strict_types=1);
3+
4+
namespace Jaeger\Symfony\Tag;
5+
6+
use Jaeger\Tag\ComponentTag;
7+
8+
class SymfonyComponentTag extends ComponentTag
9+
{
10+
public function __construct()
11+
{
12+
parent::__construct('symfony');
13+
}
14+
}

src/Tag/SymfonyVersionTag.php

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
<?php
2+
declare(strict_types=1);
3+
4+
namespace Jaeger\Symfony\Tag;
5+
6+
use Jaeger\Tag\StringTag;
7+
use Symfony\Component\HttpKernel\Kernel;
8+
9+
class SymfonyVersionTag extends StringTag
10+
{
11+
public function __construct()
12+
{
13+
parent::__construct('symfony.version', Kernel::VERSION);
14+
}
15+
}

0 commit comments

Comments
 (0)