-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Snyk] Upgrade @opentelemetry/instrumentation from 0.41.1 to 0.54.2 #863
base: main
Are you sure you want to change the base?
Conversation
Snyk has created this PR to upgrade @opentelemetry/instrumentation from 0.41.1 to 0.54.2. See this package in npm: @opentelemetry/instrumentation See this project in Snyk: https://app.snyk.io/org/sammytezzy/project/056d8995-75bd-4eb4-bca0-bc2ecbc2ba91?utm_source=github&utm_medium=referral&page=upgrade-pr
Run & review this pull request in StackBlitz Codeflow. |
|
Reviewer's Guide by SourceryThis PR upgrades the @opentelemetry/instrumentation package from version 0.41.1 to 0.54.2 to address several security vulnerabilities and incorporate the latest features and bug fixes. The upgrade spans 20 versions and includes breaking changes related to exporter configurations, HTTP instrumentation options, and internal API changes. Class diagram for changes in @opentelemetry/instrumentationclassDiagram
class ExporterConfig {
- getDefaultUrl()
- getUrlFromConfig()
- hostname
- url
- timeoutMillis
- onInit
headers: Record<string, string>
}
class InstrumentationHttp {
- ignoreIncomingPaths
- ignoreOutgoingUrls
- isIgnored
+ ignoreIncomingRequestHook()
+ ignoreOutgoingRequestHook()
}
note for ExporterConfig "Breaking changes: Removed internal methods and changed headers type"
note for InstrumentationHttp "Breaking changes: Removed deprecated options and added new hooks"
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have skipped reviewing this pull request. Here's why:
- It seems to have been created by a bot ('[Snyk]' found in title). We assume it knows what it's doing!
- We don't review packaging changes - Let us know if you'd like us to change this.
Snyk has created this PR to upgrade @opentelemetry/instrumentation from 0.41.1 to 0.54.2.
ℹ️ Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.
The recommended version is 20 versions ahead of your current version.
The recommended version was released on a month ago.
Issues fixed by the recommended upgrade:
SNYK-JS-CROSSSPAWN-8303230
SNYK-JS-IMPORTINTHEMIDDLE-5826054
SNYK-JS-FASTXMLPARSER-7573289
SNYK-JS-INFLIGHT-6095116
Release notes
Package name: @opentelemetry/instrumentation
0.54.2
🐛 (Bug Fix)
0.54.1
🐛 (Bug Fix)
0.54.0
💥 Breaking Change
getDefaultUrl
was intended for internal use has been removed from all exportersgetUrlFromConfig
was intended for internal use and has been removed from all exportershostname
was intended for internal use and has been removed from all exportersurl
was intended for internal use and has been removed from all exporterstimeoutMillis
was intended for internal use and has been removed from all exportersonInit
was intended for internal use and has been removed from all exportersheaders
type changed fromPartial<Record<string, unknown>>
toRecord<string, string>
parseHeaders
appendResourcePathToUrl
appendResourcePathToUrlIfNeeded
configureExporterTimeout
invalidTimeout
ignoreIncomingPaths
has been removed, use the more versatileignoreIncomingRequestHook
instead.ignoreOutgoingUrls
has been removed, use the more versatileignoreOutgoingRequestHook
instead.isIgnored
utility function was intended for internal use and has been removed without replacement.🚀 (Enhancement)
OTEL_SEMCONV_STABILITY_OPT_IN
containshttp
orhttp/dup
OTEL_SEMCONV_STABILITY_OPT_IN
containshttp/dup
or DOES NOT containhttp
🐛 (Bug Fix)
'none'
if env varsOTEL_EXPORTER_OTLP_TRACES_COMPRESSION
andOTEL_EXPORTER_OTLP_COMPRESSION
are falsy @ sjvans@ opentelemetry/api-logs@0.52.0
,@ opentelemetry/api-events@0.52.0
packages #5013 @ pichlermarc@ opentelemetry/api-logs@0.52.0
#5013 @ pichlermarc@ opentelemetry/api-logs@0.53.0
#5013 @ pichlermarcOTEL_EXPORTER_OTLP_METRICS_COMPRESSION
OTEL_EXPORTER_OTLP_LOGS_COMPRESSION
OTEL_EXPORTER_OTLP_METRICS_CLIENT_CERTIFICATE
OTEL_EXPORTER_OTLP_LOGS_CLIENT_CERTIFICATE
OTEL_EXPORTER_OTLP_METRICS_CLIENT_KEY
OTEL_EXPORTER_OTLP_LOGS_CLIENT_KEY
OTEL_EXPORTER_OTLP_METRICS_INSECURE
OTEL_EXPORTER_OTLP_LOGS_INSECURE
http.get
,http.request
,https.get
, andhttps.request
when used from ESM code and imported via theimport defaultExport from 'http'
style. #5024 @ trentm🏠 (Internal)
refactor(exporter-prometheus): replace
MetricAttributes
andMetricAttributeValues
withAttributes
andAttributeValues
#4993refactor(browser-detector): replace
ResourceAttributes
withAttributes
#5004refactor(sdk-logs): replace
ResourceAttributes
withAttributes
#5005 @ david-lunaImportant
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
Summary by Sourcery
Build: