Conversation
Picks up OTel instrumentation (distributed tracing, log export) from flagsmith-common and allows the W3C trace propagation headers (traceparent, tracestate, baggage) through CORS so browser-based SDKs can forward distributed trace context to the API. beep boop
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
Docker builds report
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7132 +/- ##
==========================================
- Coverage 98.34% 98.26% -0.09%
==========================================
Files 1336 1336
Lines 50128 49966 -162
==========================================
- Hits 49298 49098 -200
- Misses 830 868 +38 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
|
docs/if required so people know about the feature.Changes
Contributes to Flagsmith/flagsmith-common#182.
Bumps
flagsmith-commonfrom 3.5.0 to 3.6.0, which includes OTel instrumentation support.Adds
traceparent,tracestate, andbaggageto the defaultFLAGSMITH_CORS_EXTRA_ALLOW_HEADERSso browser-based SDKs can forward W3C distributed trace context to the API.How did you test this code?
Verified the lock file resolves cleanly and pre-commit hooks pass. The new OTel setup is opt-in, so no behaviour change without configuration.