[Instana] Use System.Text.Json by martincostello · Pull Request #4293 · open-telemetry/opentelemetry-dotnet-contrib (original) (raw)
added 17 commits
- Add support for configuring with options class.
- Add
net8.0andnet10.0TFMs. - Use modern C# features.
- Support batch export limit configuration and apply limits.
- Remove sync-over-async.
- Add support for custom
HttpClientcreation. - Remove redundant code.
- Fix InstanaExporterTests.
- Update README.
- Extend assertions to verify JSON payloads.
- Remove redundant code.
- Remove redundant mocks.
- Fix some StyleCop warnings.
Fix most StyleCop warnings and remove unused constants.
Add tests for TracerProviderBuilder extensions.
- Write literal
truestring instead of computing it. - Fix-up comments.
- Add TODO to remove manual JSON serialization.
Use System.Text.Json in tests instead.
Add CHANGELOG entry for TFM additions.
- Suppress
CA5399. - Update CHANGELOG with PR number.
- Tweak some XML documentation.
Move the suppression to the right place.
Address most Copilot code review comments.
Address more Copilot feedback.
Address final piece of Copilot review feedback.
The batch exporter already runs a background task to export, so use that rather than have another one just to use async.
Move entry to Unreleased.
Use System.Text.Json to serialize spans.
Add changelog entry for changes.
Copilot AI review requested due to automatic review settings
- Match version used elswhere in the repo.
- Fix some formatting.
This was referenced
May 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})