Skip to content

fix(flow-php/symfony-telemetry-bundle): optional http-foundation telemetry bridge#2460

Merged
norberttech merged 2 commits into
1.xfrom
flow-telemetry-bundle-mandatory-bridge
Jun 18, 2026
Merged

fix(flow-php/symfony-telemetry-bundle): optional http-foundation telemetry bridge#2460
norberttech merged 2 commits into
1.xfrom
flow-telemetry-bundle-mandatory-bridge

Conversation

@norberttech

Copy link
Copy Markdown
Member

Change Log


Added

  • flow-php/symfony-telemetry-bundle - inject trace context into outgoing response headers

Fixed

  • flow-php/symfony-telemetry-bundle - metric attributes render as a toggleable table row in the profiler panel

Changed

  • flow-php/symfony-telemetry-bundle - flow-php/symfony-http-foundation-telemetry-bridge is now an optional dependency; context propagation is silently disabled when absent

Removed

Deprecated

Security

…emetry bridge

- wire RequestCarrier/ResponseCarrier into the http kernel subscriber
for in/out W3C trace propagation
- move the bridge from require to require-dev/suggest
- disable context propagation gracefully when the bridge is absent
@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.12%. Comparing base (1fd2965) to head (478327a).
⚠️ Report is 1 commits behind head on 1.x.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##                1.x    #2460   +/-   ##
=========================================
  Coverage     85.12%   85.12%           
- Complexity    21226    21227    +1     
=========================================
  Files          1599     1599           
  Lines         65511    65511           
=========================================
  Hits          55767    55767           
  Misses         9744     9744           
Components Coverage Δ
etl 88.41% <ø> (ø)
cli 89.40% <ø> (ø)
lib-array-dot 81.44% <ø> (ø)
lib-azure-sdk 64.44% <ø> (ø)
lib-doctrine-dbal-bulk 93.61% <ø> (ø)
lib-filesystem 85.03% <ø> (ø)
lib-types 90.06% <ø> (ø)
lib-parquet 70.10% <ø> (ø)
lib-parquet-viewer 82.26% <ø> (ø)
lib-snappy 89.38% <ø> (ø)
lib-dremel 0.00% <ø> (ø)
lib-postgresql 88.59% <ø> (ø)
lib-telemetry 85.95% <ø> (ø)
bridge-filesystem-async-aws 92.74% <ø> (ø)
bridge-filesystem-azure 90.45% <ø> (ø)
bridge-monolog-http 96.82% <ø> (ø)
bridge-monolog-telemetry 94.11% <ø> (ø)
bridge-openapi-specification 92.07% <ø> (ø)
symfony-http-foundation 78.57% <ø> (ø)
bridge-psr18-telemetry 100.00% <ø> (ø)
bridge-psr3-telemetry 97.84% <ø> (ø)
bridge-psr7-telemetry 100.00% <ø> (ø)
bridge-telemetry-otlp 89.89% <ø> (ø)
bridge-symfony-http-foundation-telemetry 89.47% <ø> (ø)
bridge-symfony-filesystem-bundle 90.66% <ø> (ø)
bridge-symfony-filesystem-cache 98.14% <ø> (ø)
bridge-symfony-postgresql-bundle 93.83% <ø> (ø)
bridge-symfony-postgresql-cache 94.41% <ø> (ø)
bridge-symfony-postgresql-messenger 98.80% <ø> (ø)
bridge-symfony-postgresql-session 93.65% <ø> (ø)
bridge-symfony-telemetry-bundle 80.77% <100.00%> (ø)
adapter-chartjs 84.05% <ø> (ø)
adapter-csv 91.16% <ø> (ø)
adapter-doctrine 90.79% <ø> (ø)
adapter-google-sheet 99.18% <ø> (ø)
adapter-http 72.34% <ø> (ø)
adapter-json 88.63% <ø> (ø)
adapter-logger 50.00% <ø> (ø)
adapter-parquet 77.70% <ø> (ø)
adapter-text 74.13% <ø> (ø)
adapter-xml 83.40% <ø> (ø)
adapter-avro 0.00% <ø> (ø)
adapter-excel 94.21% <ø> (ø)
adapter-postgresql 91.11% <ø> (ø)
adapter-seal 85.42% <ø> (ø)
bridge-phpunit-postgresql 75.30% <ø> (ø)
bridge-phpunit-telemetry 80.08% <ø> (ø)
bridge-phpstan-types 0.00% <ø> (ø)
bridge-postgresql-valinor 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@norberttech norberttech merged commit a075e23 into 1.x Jun 18, 2026
38 checks passed
@norberttech norberttech deleted the flow-telemetry-bundle-mandatory-bridge branch June 18, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant