Skip to content

Update dependency express-rate-limit to v8.2.2 [SECURITY]#35

Merged
rtivital merged 1 commit intomasterfrom
renovate/npm-express-rate-limit-vulnerability
Mar 7, 2026
Merged

Update dependency express-rate-limit to v8.2.2 [SECURITY]#35
rtivital merged 1 commit intomasterfrom
renovate/npm-express-rate-limit-vulnerability

Conversation

@renovate
Copy link
Contributor

@renovate renovate bot commented Mar 6, 2026

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
express-rate-limit 8.2.18.2.2 age adoption passing confidence

GitHub Vulnerability Alerts

CVE-2026-30827

Summary

The default keyGenerator in express-rate-limit applies IPv6 subnet masking (/56 by default) to all addresses that net.isIPv6() returns true for. This includes IPv4-mapped IPv6 addresses (::ffff:x.x.x.x), which Node.js returns as request.ip on dual-stack servers.

Because the first 80 bits of all IPv4-mapped addresses are zero, a /56 (or any /32 to /80) subnet mask produces the same network key (::/56) for every IPv4 client. This collapses all IPv4 traffic into a single rate-limit bucket: one client exhausting the limit causes HTTP 429 for all other IPv4 clients.

Details

Root Cause

In source/ip-key-generator.ts:

export function ipKeyGenerator(ip: string, ipv6Subnet: number | false = 56) {
  if (ipv6Subnet && isIPv6(ip)) {
    return `${new Address6(`${ip}/${ipv6Subnet}`).startAddress().correctForm()}/${ipv6Subnet}`
  }
  return ip
}

net.isIPv6('::ffff:192.168.1.1') returns true, so IPv4-mapped addresses enter the subnet masking path. With a /56 prefix, the start address for any ::ffff:x.x.x.x is ::, producing the key ::/56.

Proof of Concept

const { isIPv6 } = require('net');
const { Address6 } = require('ip-address');

function ipKeyGenerator(ip, ipv6Subnet = 56) {
  if (ipv6Subnet && isIPv6(ip)) {
    return `${new Address6(`${ip}/${ipv6Subnet}`).startAddress().correctForm()}/${ipv6Subnet}`;
  }
  return ip;
}

console.log(ipKeyGenerator('::ffff:192.168.1.1', 56)); // ::/56
console.log(ipKeyGenerator('::ffff:10.0.0.1', 56));    // ::/56
console.log(ipKeyGenerator('::ffff:8.8.8.8', 56));     // ::/56
// ALL produce '::/56' — same bucket

End-to-End Validation

On a dual-stack Express server (app.listen(port, '::')), tested with Express 5.2.1:

  • request.ip for IPv4 clients is ::ffff:127.0.0.1
  • Rate limit key resolves to ::/56
  • After limit requests from any IPv4 client, all other IPv4 clients receive 429

When This Occurs

  • Node.js dual-stack servers (default on Linux when listening on ::)
  • Any environment where request.ip contains IPv4-mapped IPv6 addresses
  • Only affects the default keyGenerator (custom key generators are not affected)

Impact

  • Denial of Service: A single client can block all IPv4 traffic by exhausting the shared rate limit
  • Affects default configuration: No special options needed to trigger this

Affected Versions

All versions of express-rate-limit between v8.0.0 and v8.2.1.

Fix

This issue was fixed in commit 14e53888cdfd1b9798faf5b634c4206409e27fc4. This fix has been included in release v8.3.0, and backported to all affected minor versions in the form of releases v8.2.2, v8.1.1, and v8.0.2.


Release Notes

express-rate-limit/express-rate-limit (express-rate-limit)

v8.2.2

Compare Source


Configuration

📅 Schedule: Branch creation - "" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

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 was generated by Mend Renovate. View the repository job log.

@rtivital rtivital merged commit 5e4866c into master Mar 7, 2026
4 checks passed
@renovate renovate bot deleted the renovate/npm-express-rate-limit-vulnerability branch March 7, 2026 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant