Skip to content

Add http configuration package#383

Merged
brettmc merged 8 commits intoopen-telemetry:mainfrom
Nevay:feature/http-config
Jan 17, 2026
Merged

Add http configuration package#383
brettmc merged 8 commits intoopen-telemetry:mainfrom
Nevay:feature/http-config

Conversation

@Nevay
Copy link
Contributor

@Nevay Nevay commented Jun 5, 2025

Provides general configuration options for HTTP instrumentation packages.

Env-based configuration supports:

File-based configuration supports:

  • overriding known HTTP methods
  • overriding URI sanitization
  • enabling opt-in attributes for server and client spans

Depends on open-telemetry/opentelemetry-php#1621 to load configuration from config file or env.

@codecov
Copy link

codecov bot commented Jun 5, 2025

Codecov Report

❌ Patch coverage is 28.76712% with 104 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.54%. Comparing base (a3c02c8) to head (63264aa).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
.../Config/InstrumentationConfigurationHttpConfig.php 0.00% 67 Missing ⚠️
...src/Config/UriSanitizerRedactQueryStringValues.php 0.00% 13 Missing ⚠️
...nfigEnv/InstrumentationConfigurationHttpConfig.php 0.00% 9 Missing ⚠️
...tion/HttpConfig/src/Config/UriSanitizerDefault.php 0.00% 4 Missing ⚠️
...tpConfig/src/Config/UriSanitizerRedactUserInfo.php 0.00% 4 Missing ⚠️
...ion/HttpConfig/src/UriSanitizer/MultiSanitizer.php 81.81% 2 Missing ⚠️
...tion/HttpConfig/src/UriSanitizer/NoopSanitizer.php 0.00% 2 Missing ⚠️
...nstrumentation/HttpConfig/src/HttpClientConfig.php 0.00% 1 Missing ⚠️
src/Instrumentation/HttpConfig/src/HttpConfig.php 0.00% 1 Missing ⚠️
...nstrumentation/HttpConfig/src/HttpServerConfig.php 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #383      +/-   ##
============================================
- Coverage     83.46%   82.54%   -0.92%     
- Complexity     2330     2338       +8     
============================================
  Files           149      159      +10     
  Lines          8791     8823      +32     
============================================
- Hits           7337     7283      -54     
- Misses         1454     1540      +86     
Flag Coverage Δ
Aws 93.37% <ø> (ø)
Context/Swoole 0.00% <ø> (ø)
Exporter/Instana 49.80% <ø> (ø)
Instrumentation/AwsSdk 82.00% <ø> (ø)
Instrumentation/CakePHP 20.42% <ø> (ø)
Instrumentation/CodeIgniter 79.31% <ø> (ø)
Instrumentation/Curl 86.88% <ø> (-0.28%) ⬇️
Instrumentation/Doctrine 92.82% <ø> (ø)
Instrumentation/ExtAmqp 88.80% <ø> (ø)
Instrumentation/ExtRdKafka 86.13% <ø> (ø)
Instrumentation/Guzzle 76.25% <ø> (ø)
Instrumentation/HttpAsyncClient 78.94% <ø> (ø)
Instrumentation/HttpConfig 28.76% <28.76%> (?)
Instrumentation/IO 0.00% <ø> (ø)
Instrumentation/Laravel 71.65% <ø> (ø)
Instrumentation/MongoDB 76.84% <ø> (ø)
Instrumentation/MySqli 93.39% <ø> (ø)
Instrumentation/OpenAIPHP 86.71% <ø> (ø)
Instrumentation/PDO 85.67% <ø> (ø)
Instrumentation/PostgreSql 91.36% <ø> (ø)
Instrumentation/Psr14 77.41% <ø> (ø)
Instrumentation/Psr15 89.74% <ø> (ø)
Instrumentation/Psr16 97.43% <ø> (ø)
Instrumentation/Psr18 79.41% <ø> (ø)
Instrumentation/Psr3 67.70% <ø> (ø)
Instrumentation/Psr6 97.56% <ø> (ø)
Instrumentation/ReactPHP 99.41% <ø> (ø)
Instrumentation/Session 94.28% <ø> (ø)
Instrumentation/Slim 84.21% <ø> (ø)
Instrumentation/Yii ?
Logs/Monolog 100.00% <ø> (ø)
Propagation/CloudTrace 90.69% <ø> (ø)
Propagation/Instana 98.07% <ø> (ø)
Propagation/ServerTiming 94.73% <ø> (ø)
Propagation/TraceResponse 94.73% <ø> (ø)
ResourceDetectors/Azure 91.66% <ø> (ø)
ResourceDetectors/Container 93.02% <ø> (ø)
ResourceDetectors/DigitalOcean 100.00% <ø> (ø)
Sampler/Xray 78.38% <ø> (ø)
Shims/OpenTracing 92.99% <ø> (ø)
SqlCommenter 95.58% <ø> (ø)
Symfony 88.14% <ø> (ø)
Utils/Test 87.79% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...n/HttpConfig/src/UriSanitizer/DefaultSanitizer.php 100.00% <100.00%> (ø)
...izer/RedactSensitiveQueryStringValuesSanitizer.php 100.00% <100.00%> (ø)
...c/UriSanitizer/RedactUsernamePasswordSanitizer.php 100.00% <100.00%> (ø)
...nstrumentation/HttpConfig/src/HttpClientConfig.php 0.00% <0.00%> (ø)
src/Instrumentation/HttpConfig/src/HttpConfig.php 0.00% <0.00%> (ø)
...nstrumentation/HttpConfig/src/HttpServerConfig.php 0.00% <0.00%> (ø)
...ion/HttpConfig/src/UriSanitizer/MultiSanitizer.php 81.81% <81.81%> (ø)
...tion/HttpConfig/src/UriSanitizer/NoopSanitizer.php 0.00% <0.00%> (ø)
...tion/HttpConfig/src/Config/UriSanitizerDefault.php 0.00% <0.00%> (ø)
...tpConfig/src/Config/UriSanitizerRedactUserInfo.php 0.00% <0.00%> (ø)
... and 3 more

... and 4 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a3c02c8...63264aa. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stale
Copy link

stale bot commented Oct 18, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Oct 18, 2025
@Nevay Nevay force-pushed the feature/http-config branch from d959b69 to 0e8b9f0 Compare January 16, 2026 13:19
@Nevay Nevay marked this pull request as ready for review January 16, 2026 13:23
@Nevay Nevay requested a review from a team as a code owner January 16, 2026 13:23
@stale stale bot removed the stale label Jan 16, 2026
@ChrisLightfootWild ChrisLightfootWild requested a review from a team January 16, 2026 23:16
@brettmc brettmc merged commit 4d13696 into open-telemetry:main Jan 17, 2026
162 of 175 checks passed
@brettmc
Copy link
Contributor

brettmc commented Jan 17, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants