Skip to content

Conversation

@vordimous
Copy link
Contributor

Which problem is this PR solving?

  • I spent a few days trying to figure out how to stop duplicate logs from happening while needing to use the OpenTelemetryTransportV3 to format my payload correctly.

Short description of the changes

  • This adds some clarity and an example how to correctly disable the default logger in the auto instrumentation.

@vordimous vordimous requested a review from a team as a code owner May 18, 2025 17:54
@github-actions github-actions bot requested a review from seemk May 18, 2025 17:54
@pichlermarc pichlermarc requested a review from trentm July 2, 2025 16:51
@vordimous vordimous requested a review from trentm July 10, 2025 12:39
@codecov
Copy link

codecov bot commented Jul 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.07%. Comparing base (4384c7b) to head (3c147e2).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2831   +/-   ##
=======================================
  Coverage   89.07%   89.07%           
=======================================
  Files         188      188           
  Lines        9223     9223           
  Branches     1900     1900           
=======================================
  Hits         8215     8215           
  Misses       1008     1008           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@trentm trentm changed the title feat: Clarify the duplicate logging workaround docs(instrumentation-winston): Clarify the duplicate logging workaround Jul 10, 2025
@trentm trentm merged commit de22600 into open-telemetry:main Jul 10, 2025
24 checks passed
@trentm
Copy link
Contributor

trentm commented Jul 10, 2025

Thanks, @vordimous!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants