Skip to content

Commit 475693e

Browse files
committed
next bunch of psalm fixes
1 parent 8aa437e commit 475693e

File tree

7 files changed

+19
-5
lines changed

7 files changed

+19
-5
lines changed

src/Instrumentation/PDO/src/PDOInstrumentation.php

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
use OpenTelemetry\API\Trace\SpanKind;
1111
use OpenTelemetry\API\Trace\StatusCode;
1212
use OpenTelemetry\Context\Context;
13-
use OpenTelemetry\SDK\Common\Configuration\Configuration;
1413
use function OpenTelemetry\Instrumentation\hook;
14+
use OpenTelemetry\SDK\Common\Configuration\Configuration;
1515
use OpenTelemetry\SemConv\TraceAttributes;
1616
use OpenTelemetry\SemConv\Version;
1717
use PDO;
@@ -203,7 +203,8 @@ public static function register(): void
203203
pre: static function (PDOStatement $statement, array $params, string $class, string $function, ?string $filename, ?int $lineno) use ($pdoTracker, $instrumentation) {
204204
$attributes = $pdoTracker->trackedAttributesForStatement($statement);
205205
if (self::isDistributeStatementToLinkedSpansEnabled()) {
206-
$attributes[TraceAttributes::DB_STATEMENT] = $statement->queryString;
206+
/** @psalm-suppress InvalidArrayAssignment */
207+
$attributes[TraceAttributes::DB_QUERY_TEXT] = $statement->queryString;
207208
}
208209
/** @psalm-suppress ArgumentTypeCoercion */
209210
$builder = self::makeBuilder($instrumentation, 'PDOStatement::fetchAll', $function, $class, $filename, $lineno)
@@ -228,9 +229,10 @@ public static function register(): void
228229
$attributes = $pdoTracker->trackedAttributesForStatement($statement);
229230

230231
if (self::isDistributeStatementToLinkedSpansEnabled()) {
231-
$attributes[TraceAttributes::DB_STATEMENT] = $statement->queryString;
232+
/** @psalm-suppress InvalidArrayAssignment */
233+
$attributes[TraceAttributes::DB_QUERY_TEXT] = $statement->queryString;
232234
}
233-
235+
234236
/** @psalm-suppress ArgumentTypeCoercion */
235237
$builder = self::makeBuilder($instrumentation, 'PDOStatement::execute', $function, $class, $filename, $lineno)
236238
->setSpanKind(SpanKind::KIND_CLIENT)
@@ -285,6 +287,6 @@ private static function isDistributeStatementToLinkedSpansEnabled(): bool
285287
return Configuration::getBoolean('OTEL_PHP_INSTRUMENTATION_PDO_DISTRIBUTE_STATEMENT_TO_LINKED_SPANS', false);
286288
}
287289

288-
return get_cfg_var('otel.instrumentation.pdo.distribute_statement_to_linked_spans');
290+
return filter_var(get_cfg_var('otel.instrumentation.pdo.distribute_statement_to_linked_spans'), FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE) ?? false;
289291
}
290292
}

src/Sampler/RuleBased/src/ComponentProvider/SamplerRuleBased.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,8 @@ public function createPlugin(array $properties, Context $context): SamplerInterf
5252
public function getConfig(ComponentProviderRegistry $registry): ArrayNodeDefinition
5353
{
5454
$node = new ArrayNodeDefinition('contrib_rule_based');
55+
56+
/** @psalm-suppress PossiblyNullReference */
5557
$node
5658
->children()
5759
->arrayNode('rule_sets')

src/Sampler/RuleBased/src/ComponentProvider/SamplingRuleAttribute.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ public function createPlugin(array $properties, Context $context): SamplingRule
3636
public function getConfig(ComponentProviderRegistry $registry): ArrayNodeDefinition
3737
{
3838
$node = new ArrayNodeDefinition('attribute');
39+
40+
/** @psalm-suppress PossiblyNullReference */
3941
$node
4042
->children()
4143
->scalarNode('key')

src/Sampler/RuleBased/src/ComponentProvider/SamplingRuleLink.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@ public function createPlugin(array $properties, Context $context): SamplingRule
3434
public function getConfig(ComponentProviderRegistry $registry): ArrayNodeDefinition
3535
{
3636
$node = new ArrayNodeDefinition('link');
37+
38+
/** @psalm-suppress PossiblyNullReference */
3739
$node
3840
->children()
3941
->booleanNode('sampled')->isRequired()->end()

src/Sampler/RuleBased/src/ComponentProvider/SamplingRuleParent.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@ public function createPlugin(array $properties, Context $context): SamplingRule
3434
public function getConfig(ComponentProviderRegistry $registry): ArrayNodeDefinition
3535
{
3636
$node = new ArrayNodeDefinition('parent');
37+
38+
/** @psalm-suppress PossiblyNullReference */
3739
$node
3840
->children()
3941
->booleanNode('sampled')->isRequired()->end()

src/Sampler/RuleBased/src/ComponentProvider/SamplingRuleSpanKind.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,8 @@ public function createPlugin(array $properties, Context $context): SamplingRule
3939
public function getConfig(ComponentProviderRegistry $registry): ArrayNodeDefinition
4040
{
4141
$node = new ArrayNodeDefinition('span_kind');
42+
43+
/** @psalm-suppress PossiblyNullReference */
4244
$node
4345
->children()
4446
->enumNode('kind')

src/Sampler/RuleBased/src/ComponentProvider/SamplingRuleSpanName.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@ public function createPlugin(array $properties, Context $context): SamplingRule
3434
public function getConfig(ComponentProviderRegistry $registry): ArrayNodeDefinition
3535
{
3636
$node = new ArrayNodeDefinition('span_name');
37+
38+
/** @psalm-suppress PossiblyNullReference */
3739
$node
3840
->children()
3941
->scalarNode('pattern')

0 commit comments

Comments
 (0)