This repository was archived by the owner on Mar 17, 2026. It is now read-only.
Commit aedc36c
fix: added support for Opentelemetry 0.18 (#1234)
* fix: added support for Opentelemetry 0.18
Introduces support for Opentelemetry 0.18 API which implements the
Tracing 1.0 specification.
* style: remove unused import declaration
* refactor: removed the `OpenTelemetryTracer`-class and just expose createSpan as function
* test: updated the span related tests
* test: updated the tests
* test: updated the tests
* refactor: move the tracer out of the `createSpan`-function
Small refactor to ensure no new tracer gets created for each call to the
`createSpan`-functio by initiating the `libraryTracer`-variable
* fix: get the unit test for publisher opentelemetry working again
* test: fix the subscriber unit tests for opentelemetry
* chore: remove describe.only
* chore: latest changes
* test: updated the tests
* test: improve the tests
* chore: maybe its need to be Google LLC?
* feat: add messaging attributes to otel spans
* fix: include package.json version as instrumentation version
* docs: update the opentelemetry example code
* style: remove unused code
* fix: remove peer name for now
* style: improve explanation regarding otel trace provider
* chore: update the opentelemetry versios in the samples `package.json`
* test: remove extranous logging in the opentelemetry example app
* test: remove the check for `traceId` that doesn't get outputted
* fix: remove unnecessary messaging span attribute and added comments
* style: use shiny new syntax for undefined check in `subscriber`-class
* fix: match the span name with the operation kind in `subscriber`-class
* chore: improved code based on PR feedback
* fix:
BREAKING CHANGE:
The Opentelemetry tracing span doesn't expose the `data`-attribute anymore
to avoid accidentally exposing privacy/personal data
* fix: only include `googclient_OpenTelemetrySpanContext`-attribute
when valid span context exists
* fix: change the way `package.json` is being imported
* fix: revert bad merge that removed tracing on devDependencies
At least I think that's what happened...
* style: remove unnecessary comment
* style: remove unused imports
* build: downgrade @sinonjs/fake-timers to v6
The v10 of sinonjs comes with v7 of fake-timers which comes with
typescript type definitions that are generated by jsdoc comments which
seems to cause trouble when compiling the app it.
Downgrading to v6 to avoid needing to solve this problem now.
* style: cleanup type definition
Removed the `| undefined` for the `span` in the `Publisher`-class
Co-authored-by: Aaron Abbott <[email protected]>
* build: remove `package.json` after compilation step
* build: put back package.json in the build
* build: explicitly add package.json to the npm pack build
* fix: revert "fix: change the way `package.json` is being imported"
This reverts commit 0c2c76f.
* fix: revert "build: explicitly add package.json to the npm pack build"
This reverts commit 14de625.
Co-authored-by: Weyert de Boer <[email protected]>
Co-authored-by: Megan Potter <[email protected]>
Co-authored-by: Megan Potter <[email protected]>
Co-authored-by: Aaron Abbott <[email protected]>1 parent e85b34c commit aedc36c
14 files changed
Lines changed: 1328 additions & 176 deletions
File tree
- protos
- samples
- system-test
- src
- publisher
- test
- publisher
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
56 | | - | |
57 | | - | |
| 56 | + | |
| 57 | + | |
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
| |||
69 | 69 | | |
70 | 70 | | |
71 | 71 | | |
| 72 | + | |
72 | 73 | | |
73 | 74 | | |
74 | 75 | | |
| |||
77 | 78 | | |
78 | 79 | | |
79 | 80 | | |
80 | | - | |
| 81 | + | |
81 | 82 | | |
82 | 83 | | |
83 | 84 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments