Skip to content

fix: fix defaultInstrumentation never being registered bug#59

Merged
shubham-sumo merged 2 commits intomainfrom
default-Instrumentation
Feb 4, 2026
Merged

fix: fix defaultInstrumentation never being registered bug#59
shubham-sumo merged 2 commits intomainfrom
default-Instrumentation

Conversation

@shubham-sumo
Copy link
Contributor

Description:
Bug Fixes:

  • HttpInstrumentation , RedisInstrumentation , MongoDBInstrumentation and other default instrumentations were never registered because the condition typeof configureInstrumentations === "function" was always true (since global.configureInstrumentations returns []), causing defaultConfigureInstrumentations() to never be called.

Testing: #58 screenshots attached

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@shubham-sumo shubham-sumo marked this pull request as ready for review February 3, 2026 05:34
@shubham-sumo shubham-sumo requested a review from a team as a code owner February 3, 2026 05:34
Copy link
Contributor

@raunakmandaokar raunakmandaokar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Add Integration-tests

@shubham-sumo shubham-sumo merged commit 13e7b76 into main Feb 4, 2026
20 checks passed
@shubham-sumo shubham-sumo deleted the default-Instrumentation branch February 4, 2026 09:56
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.

2 participants