Skip to content
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.

Sign up for GitHub

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

feat(lru-memoizer): add instrumentation for lru-memoizer #1056

Merged
merged 12 commits into from
Jun 17, 2022
Merged

feat(lru-memoizer): add instrumentation for lru-memoizer #1056

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
9714594
feat(lru-memoizer): add instrumentation for lru-memoizer
Jun 13, 2022
ee24f16
fix(lru-memoizer): typescript issues in test
Jun 14, 2022
f582cad
fix(auto-instrumentation): fix tests
Jun 14, 2022
eadcfc2
chore: downgrade expect to support node 8
Jun 14, 2022
49c3c23
chore(lru-memoizer): fix expect version
Jun 14, 2022
43ac6fc
fix(lru-memoizer): typos and rephrase
Jun 14, 2022
3138bd2
chore(lru-memoizer): comments rephrases
Jun 14, 2022
06ef6c7
style(lru-memoizer): unified suppored versions
Jun 14, 2022
08b92cc
style(lru-memoizer): better wording in README
Jun 14, 2022
6006e7e
fix(lru-memoizer): check cb before binding
Jun 15, 2022
f2c3569
Merge branch 'main' into lru-memoize
rauno56 Jun 15, 2022
8d8211e
Merge branch 'main' into lru-memoize
rauno56 Jun 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
style(lru-memoizer): better wording in README
  • Loading branch information
Amir Blum committed Jun 14, 2022
commit 08b92cce5f4f13eac3ebc11fd07f4c64c0b13667
2 changes: 1 addition & 1 deletion plugins/node/instrumentation-lru-memoizer/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ npm install --save @opentelemetry/instrumentation-lru-memoizer

## Usage

This instrumentation does not produce any telemetry data. It only bind the caller context to callbacks so downstream operations are recorded with the right context (traceId / parentSpanId / baggage / etc). This package is a dependency for other popular packages such as [jwks-rsa](https://www.npmjs.com/package/jwks-rsa)
This instrumentation does not produce any telemetry data. It only bind the caller context to callbacks so downstream operations are recorded with the right context (traceId / parentSpanId / baggage / etc). The `lru-memoizer` package is a dependency for other packages such as [jwks-rsa](https://www.npmjs.com/package/jwks-rsa)

To load a specific plugin, specify it in the registerInstrumentations's configuration:

Expand Down