Skip to content

Update npm package @opentelemetry/sdk-node to v0.217.0 [SECURITY]#8739

Open
hash-worker[bot] wants to merge 1 commit into
mainfrom
deps/js/npm-opentelemetry-sdk-node-vulnerability
Open

Update npm package @opentelemetry/sdk-node to v0.217.0 [SECURITY]#8739
hash-worker[bot] wants to merge 1 commit into
mainfrom
deps/js/npm-opentelemetry-sdk-node-vulnerability

Conversation

@hash-worker
Copy link
Copy Markdown
Contributor

@hash-worker hash-worker Bot commented May 22, 2026

This PR contains the following updates:

Package Change Age Confidence
@opentelemetry/sdk-node (source) 0.207.00.217.0 age confidence

Warning

Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

GitHub Vulnerability Alerts

CVE-2026-44902

Summary

A single malformed HTTP request crashes any Node.js process running the OpenTelemetry JS Prometheus exporter. The metrics endpoint (default 0.0.0.0:9464) has no error handling around URL parsing, so a request with an invalid URI causes an uncaught TypeError that terminates the process.

You are affected by this vulnerability if either of the following apply to your application:

  • you directly use @opentelemetry/exporter-prometheus in your code through its built-in server.
  • your OTEL_METRICS_EXPORTER environment variable includes prometheus AND
    • you use @opentelemetry/sdk-node
    • you use @opentelemetry/auto-instrumentations-node via --require @​opentelemetry/auto-instrumentations-node/register/--import @​opentelemetry/auto-instrumentations-node/register

Impact

Denial of service. Any application using the OpenTelemetry Prometheus exporter’s built-in server can be crashed by a single unauthenticated network packet sent to the metrics port. No authentication, special privileges, or prior access is required.

Remediation

Update to the fixed version

Update @opentelemetry/exporter-prometheus and @opentelemetry/sdk-node to version 0.217.0 or later.
Update @opentelemetry/auto-instrumentations-node to version 0.75.0 or later.

This release adds proper error handling around the URL constructor, returning an HTTP 400 response on parse failure rather than allowing the exception to propagate and crash the process.

npm install @​opentelemetry/exporter-prometheus@latest

Do Not Expose the Endpoint to Untrusted Users

Important

The following mitigations reduce exposure but do not fully remediate the vulnerability. Any client that can reach the metrics endpoint - including your own Prometheus scraper host if compromised - could still trigger the crash. Updating to 0.217.0 is the recommended resolution.

If updating is not immediately feasible, restrict access to the metrics endpoint so that it is not reachable by untrusted or unauthenticated network clients. For example:

  • Bind to localhost only by setting the host option to 127.0.0.1 when configuring the PrometheusExporter, so the port is not exposed on public or shared network interfaces

  • Use a firewall or network policy to restrict access to port 9464 (or whichever port you have configured) to only trusted Prometheus scrape hosts

  • Place the endpoint behind a reverse proxy that filters or validates incoming requests before they reach the exporter

Details

In PrometheusExporter.ts, the _requestHandler calls new URL(request.url, this._baseUrl) without any error handling. Node's HTTP parser accepts absolute-form URIs (e.g. http://) for proxy compatibility, including malformed ones. When request.url is "http://", the URL constructor throws TypeError: Invalid URL. Since there is no try-catch in the handler, the exception propagates as an uncaught exception and crashes the process.

The Prometheus metrics endpoint is unauthenticated by design (Prometheus scrapes it) and binds to 0.0.0.0 by default, meaning it is reachable by any network client that can connect to the metrics port.

Proof of Concept

Start any Node.js application with the Prometheus exporter running on the default port 9464, then send a single raw TCP packet:

echo -ne 'GET http:// HTTP/1.1\r\nHost: localhost\r\n\r\n' | nc localhost 9464

The process crashes immediately with:

TypeError: Invalid URL
    at new URL (...)
    at PrometheusExporter._requestHandler (...)

Release Notes

open-telemetry/opentelemetry-js (@​opentelemetry/sdk-node)

v0.217.0

Compare Source

v0.216.0

Compare Source

v0.215.0

Compare Source

v0.214.0

Compare Source

v0.213.0

Compare Source

v0.212.0

Compare Source

v0.211.0

Compare Source

v0.210.0

Compare Source

v0.209.0

Compare Source

v0.208.0

Compare Source


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • ""
  • Automerge
    • "before 4am every weekday,every weekend"

🚦 Automerge: Enabled.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@hash-worker hash-worker Bot enabled auto-merge May 22, 2026 12:24
@hash-worker
Copy link
Copy Markdown
Contributor Author

hash-worker Bot commented May 22, 2026

⚠️ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you click the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: yarn.lock
error This project's package.json defines "packageManager": "yarn@4.12.0". However the current global version of Yarn is 1.22.22.

Presence of the "packageManager" field indicates that the project is meant to be used with Corepack, a tool included by default with all official Node.js distributions starting from 16.9 and 14.19.
Corepack must currently be enabled by running corepack enable in your terminal. For more information, check out https://yarnpkg.com/corepack.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hash Error Error May 22, 2026 12:25pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
hashdotdesign-tokens Ignored Ignored Preview May 22, 2026 12:25pm
petrinaut Skipped Skipped May 22, 2026 12:25pm

@cursor
Copy link
Copy Markdown

cursor Bot commented May 22, 2026

PR Summary

Low Risk
Low risk dependency bump limited to the tests/hash-backend-load package; potential risk is minor runtime behavior changes in OpenTelemetry instrumentation/exporter initialization during load tests.

Overview
Updates @opentelemetry/sdk-node from 0.207.0 to 0.217.0 in the tests/hash-backend-load package to pick up the latest OpenTelemetry Node SDK (including security fixes). No application or test logic changes beyond the dependency version bump.

Reviewed by Cursor Bugbot for commit e28affb. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions github-actions Bot added area/deps Relates to third-party dependencies (area) area/tests New or updated tests labels May 22, 2026
@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented May 22, 2026

🤖 Augment PR Summary

Summary: Updates the load-test package to use @opentelemetry/sdk-node v0.217.0.

Why: Picks up the security fix called out in the GitHub advisory (CVE-2026-44902) for Prometheus exporter URL parsing crash scenarios.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

"@opentelemetry/exporter-trace-otlp-grpc": "0.207.0",
"@opentelemetry/resources": "2.2.0",
"@opentelemetry/sdk-node": "0.207.0",
"@opentelemetry/sdk-node": "0.217.0",
Copy link
Copy Markdown

@augmentcode augmentcode Bot May 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tests/hash-backend-load/package.json:41 — This bumps @opentelemetry/sdk-node to 0.217.0 while @opentelemetry/exporter-trace-otlp-grpc remains at 0.207.0; since these 0.x OpenTelemetry packages tend to be version-aligned, this can result in duplicate/mismatched internal OTel dependencies and subtle runtime issues. Consider aligning the OpenTelemetry experimental package versions (or double-checking peer/interop compatibility for this combo).

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e28affb. Configure here.

"@opentelemetry/exporter-trace-otlp-grpc": "0.207.0",
"@opentelemetry/resources": "2.2.0",
"@opentelemetry/sdk-node": "0.207.0",
"@opentelemetry/sdk-node": "0.217.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OpenTelemetry experimental packages version mismatch after partial upgrade

Medium Severity

@opentelemetry/sdk-node is bumped to 0.217.0 while @opentelemetry/exporter-trace-otlp-grpc remains at 0.207.0. These are experimental packages from the same monorepo released in lockstep and intended to be used at matching versions. The 10-version gap can cause duplicate internal dependency instances (e.g., @opentelemetry/resources at both 2.2.0 and ~2.7.x), potentially breaking context propagation or resource merging in the tracing setup.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e28affb. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/deps Relates to third-party dependencies (area) area/tests New or updated tests

Development

Successfully merging this pull request may close these issues.

1 participant