-
Notifications
You must be signed in to change notification settings - Fork 792
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ZipkinExporter] ExporterConfig needs to be exported since it is API #1453
Labels
bug
Something isn't working
Comments
I would like to work on this, please assign to me |
pichlermarc
pushed a commit
to dynatrace-oss-contrib/opentelemetry-js
that referenced
this issue
Dec 15, 2023
…ng:true (open-telemetry#1453) * fix(pg-values): values should be parsable when enhancedDatabaseReporting:true * fix(pg-values): add try catch * fix(pg-values): lint * fix(pg-values): fixes * fix(pg-values): fix * Update plugins/node/opentelemetry-instrumentation-pg/src/utils.ts Co-authored-by: Gerhard Stöbich <[email protected]> * fix(pg-values): support buffer, object, null and undefined * Update plugins/node/opentelemetry-instrumentation-pg/src/utils.ts Co-authored-by: Gerhard Stöbich <[email protected]> * fix(pg-values): fixes * fix(pg-values): lint * fix(pg-values): add test for 'toPostgres' * fix(pg-values): fix --------- Co-authored-by: Gerhard Stöbich <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Please answer these questions before submitting a bug report.
What version of OpenTelemetry are you using?
0.10.2
What version of Node are you using?
14.8.0
What did you do?
import { ExporterConfig } from '@opentelemetry/exporter-zipkin';
What did you expect to see?
No errors
What did you see instead?
Module '"../node_modules/@opentelemetry/exporter-zipkin/build/src"' has no exported member 'ExporterConfig'.
Additional context
The ZipkinExporter constructor requires the type ExporterConfig which is not exported from the module. It should be exported since it is part of the API.
The text was updated successfully, but these errors were encountered: