Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: open-telemetry/opentelemetry-js-contrib
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: instrumentation-mongodb-v0.37.3
Choose a base ref
...
head repository: open-telemetry/opentelemetry-js-contrib
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: instrumentation-mongodb-v0.38.0
Choose a head ref
  • 19 commits
  • 90 files changed
  • 16 contributors

Commits on Nov 22, 2023

  1. Configuration menu
    Copy the full SHA
    71172e5 View commit details
    Browse the repository at this point in the history

Commits on Nov 27, 2023

  1. fix(instrumentation-net): Don't operate on closed span (#1819)

    * fix(instrumentation-net): Don't operate on closed span
    
    * Fix lint
    mhennoch authored Nov 27, 2023
    Configuration menu
    Copy the full SHA
    60d60d0 View commit details
    Browse the repository at this point in the history

Commits on Nov 28, 2023

  1. feat(instrumentation-bunyan): add log sending to Logs Bridge API (#1713)

    * feat(instrumentation-bunyan): add log sending to Logs Bridge API
    
    This extends the Bunyan instrumentation to automatically add a
    Bunyan stream to created loggers that will send log records to
    the Logs Bridge API: https://opentelemetry.io/docs/specs/otel/logs/bridge-api/
    
    Now that the instrumentation supports separate "injection" of fields
    and "bridging" of log records functionality, this also adds two boolean
    options to disable those independently: `enableInjection` and
    `enableLogsBridge`.
    
    This also updates the instrumentation to work with ES module usage.
    
    Closes: #1559
    
    * markdown lint fixes
    
    * markdown lint fixes
    
    * catch up with recent core-deps update
    
    * some type tweaks suggested by David
    
    * more specific type
    
    Co-authored-by: Amir Blum <[email protected]>
    
    * use more self-explanatory code for mapping Bunyan level to OTel severity, from blumamir
    
    * export OpenTelemetryBunyanStream for direct usage in Bunyan loggers without the instrumentation
    
    * .apply over .call suggestion
    
    * consistency suggestion
    
    * suggestion to use the longer (perhaps clearer) logger var name
    
    * switch to false-by-default config vars to avoid surprises with undefined values
    
    See https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/configuration/sdk-environment-variables.md#boolean-value
    Suggestion from blumamir.
    
    * document using OpenTelemetryBunyanStream without the instrumentation
    
    * fix https://eslint.org/docs/latest/rules/prefer-spread lint error
    
    * drop options to OpenTelemetryBunyanStream constructor because YAGNI
    
    * temporarily drop CI caching to test theory on unit-test (18) CI failure
    
    * more CI debugging: restore cache, add some 'npm ls -a' to look into NoopContextManager being used
    
    * elide Bunyan 'pid' and 'hostname' fields in OTel log record attributes
    
    Because they are redundant with 'process.pid' and 'host.name'
    resource attributes. Add some docs on how to use resource detectors
    to the example, because the HostDetector is not on by default in
    the NodeSDK.
    
    * update test for having elided 'pid' and 'hostname' fields
    
    * CI debugging: ignore the 'npm ls -a' exit status, they shouldn't break the build
    
    * fix lint and compile errors
    
    * CI debugging: turn on diag DEBUG to test a theory
    
    * turn off diag in this example
    
    * undo CI debugging changes
    
    * update deps to current releases and sync package-lock.json
    
    * disableInjection -> disableLogCorrelation
    
    * disableLogsBridge -> disableLogSending
    
    Avoid using 'bridge' terminology at suggestion from specs that the Bridge API is an internal detail.
    
    * correct the default instrumentation scope name (as discussed earlier)
    
    * tests: fix test for intrumentationScope.name change in previous commit
    
    * fix lint
    
    ---------
    
    Co-authored-by: Amir Blum <[email protected]>
    Co-authored-by: Hector Hernandez <[email protected]>
    3 people authored Nov 28, 2023
    Configuration menu
    Copy the full SHA
    4a1d83c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    f74bcc2 View commit details
    Browse the repository at this point in the history
  3. feat(instrumentation-mongodb): add support for mongodb v6 (#1760)

    * feat: add support for mongodb v6
    
    * chore: lint fix
    
    * chore: revert bump of mongodb
    
    * chore: revert test script
    
    * chore: revert changelog
    
    * chore: filter node versions in TAV
    
    * chore: fix .tav.yml
    
    ---------
    
    Co-authored-by: Daniel Dyla <[email protected]>
    david-luna and dyladan authored Nov 28, 2023
    Configuration menu
    Copy the full SHA
    660e37b View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    b39c96c View commit details
    Browse the repository at this point in the history

Commits on Nov 29, 2023

  1. fix(instrumentation-fastify): fix span attributes and avoid FSTDEP017…

    … FastifyDeprecation warning for 404 request (#1763)
    
    For a 404 `request.routeOptions.url` is undefined. Since [email protected]
    when routeOptions was added, we shouldn't fallback to the deprecated
    request.routerPath.
    
    This also corrects the assumption that the handler name is "bound ..."
    in all cases. E.g. for a 404 it is Fastify's core "basic404" internal
    function.
    
    Also add a test that Fastify instrumentation works for ESM usage.
    
    Fixes: #1757
    Co-authored-by: Marc Pichler <[email protected]>
    trentm and pichlermarc authored Nov 29, 2023
    Configuration menu
    Copy the full SHA
    18ae75c View commit details
    Browse the repository at this point in the history

Commits on Nov 30, 2023

  1. Configuration menu
    Copy the full SHA
    4ca1862 View commit details
    Browse the repository at this point in the history

Commits on Dec 1, 2023

  1. feat(cucumber): support @cucumber/cucumber@10 (#1830)

    Co-authored-by: Daniel Dyla <[email protected]>
    Ugzuzg and dyladan authored Dec 1, 2023
    Configuration menu
    Copy the full SHA
    1c2e8b2 View commit details
    Browse the repository at this point in the history
  2. fix(instrumentation-lambda): soften "unable to init" message and demo…

    …te to diag.debug (#1836)
    
    
    Co-authored-by: Trent Mick <[email protected]>
    beggers and trentm authored Dec 1, 2023
    Configuration menu
    Copy the full SHA
    fb80783 View commit details
    Browse the repository at this point in the history

Commits on Dec 4, 2023

  1. docs(user-interaction): update docs to include examples of config opt…

    …ions (#1840)
    
    * docs(user-interaction): update docs to include examples of config options
    
    * add example to prevent span creation
    
    * fix lint
    
    * address PR comments
    
    * update link to list of events
    
    * syntax updates
    
    Co-authored-by: Martin Kuba <[email protected]>
    
    ---------
    
    Co-authored-by: Martin Kuba <[email protected]>
    pkanal and martinkuba authored Dec 4, 2023
    Configuration menu
    Copy the full SHA
    1c24cfd View commit details
    Browse the repository at this point in the history

Commits on Dec 6, 2023

  1. test(instrumentation-aws-sdk): fix SQS mock responses for @aws-sdk/cl…

    …ient-sqs versions using the AWS JSON 1.0 protocol (#1844)
    
    Versions 3.446.0 and later switched to a new JSON protocol.
    https://github.com/aws/aws-sdk-js-v3/blob/main/clients/client-sqs/CHANGELOG.md#34460-2023-11-08
    
    Refs: #1838 (comment)
    trentm authored Dec 6, 2023
    Configuration menu
    Copy the full SHA
    094fd6f View commit details
    Browse the repository at this point in the history
  2. fix(resource-detector-azure): only detect Azure Functions when FUNCTI…

    …ONS_EXTENSION_VERSION is defined (#1846)
    JacksonWeber authored Dec 6, 2023
    Configuration menu
    Copy the full SHA
    8f2a195 View commit details
    Browse the repository at this point in the history

Commits on Dec 7, 2023

  1. Configuration menu
    Copy the full SHA
    b61f912 View commit details
    Browse the repository at this point in the history
  2. fix(instrumentation-pino): instrument pino used in ESM (#1831)

    This also reduces the number of pino versions tested with
    test-all-versions test (from 42 to 13, currently).
    trentm authored Dec 7, 2023
    Configuration menu
    Copy the full SHA
    4782f5b View commit details
    Browse the repository at this point in the history
  3. feat(host-metrics)!: use the package name as the default instrumentat…

    …ion scope name, to align with instrumentations (#1822)
    
    Closes: #1782
    trentm authored Dec 7, 2023
    Configuration menu
    Copy the full SHA
    bcf3501 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    86a21d7 View commit details
    Browse the repository at this point in the history
  5. fix(instrumentation-aws-sdk): remove un-sanitised db.statement span a…

    …ttribute from DynamoDB spans (#1748)
    
    * feat: add configuration to customise dynamodb statement serialization
    
    * fix: format readme
    
    * chore: pass DiagLogger to DynamoDB instrumentation
    
    * chore: omit db statement when serializer is not configured or when it returned undefined
    Allow for passing operation to serializer
    
    * chore: run lint:fix
    
    ---------
    
    Co-authored-by: Amir Blum <[email protected]>
    ramesius and blumamir authored Dec 7, 2023
    Configuration menu
    Copy the full SHA
    cdbb29f View commit details
    Browse the repository at this point in the history
  6. chore: release main (#1829)

    * chore: release main
    
    * chore: sync package-lock.json
    
    ---------
    
    Co-authored-by: opentelemetrybot <[email protected]>
    dyladan and opentelemetrybot authored Dec 7, 2023
    Configuration menu
    Copy the full SHA
    c7e7000 View commit details
    Browse the repository at this point in the history
Loading