Skip to content

chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.2 [security] - #215

Open
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/npm-markdown-it-vulnerability
Open

renovate[bot] wants to merge 1 commit into
mainfrom
renovate/npm-markdown-it-vulnerability

Conversation

@renovate

@renovate renovate Bot commented Apr 9, 2026 •

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
markdown-it >=14.1.0 → >=14.3.2 age confidence

markdown-it is has a Regular Expression Denial of Service (ReDoS)

CVE-2026-2327 / GHSA-38c4-r59v-3vqw

More information

Details

Versions of the package markdown-it from 13.0.0 and before 14.1.1 are vulnerable to Regular Expression Denial of Service (ReDoS) due to the use of the regex /*+$/ in the linkify function. An attacker can supply a long sequence of * characters followed by a non-matching character, which triggers excessive backtracking and may lead to a denial-of-service condition.

Severity

  • CVSS Score: 5.5 / 10 (Medium)
  • Vector String: CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:L/SC:N/SI:N/SA:N/E:P

References

This data is provided by the GitHub Advisory Database (CC-BY 4.0).


markdown-it: Quadratic complexity DoS in smartquotes rule via replaceAt string operations

CVE-2026-48988 / GHSA-6v5v-wf23-fmfq

More information

Details

Summary

A quadratic time complexity vulnerability exists in markdown-it's smartquotes rule (enabled via the typographer: true option). An attacker can craft a markdown input consisting of consecutive quotation marks that causes the parser to consume excessive CPU time, leading to denial of service.

Details

The vulnerability is in the replaceAt() helper function used by the smartquotes rule in lib/rules_core/smartquotes.mjs:

function replaceAt (str, index, ch) {
  return str.slice(0, index) + ch + str.slice(index + 1)
}

When markdown-it processes a text token containing many quotation marks (either " or ') with typographer: true, the smartquotes rule iterates through each quote character and calls replaceAt() to substitute it with a typographic (curly) quote. Each call to replaceAt() creates three new string slices and concatenates them, which is an O(n) operation where n is the length of the string.

Since this is called once per quote character in the token, and there are n quote characters, the total time complexity becomes O(n^2).

The root cause is that the smartquotes rule modifies token.content in place using string slicing rather than building the result incrementally. The process_inlines() function (line 14) processes each quote in the text token, and for matching quote pairs, calls replaceAt() on both the opening and closing token's content (lines 151-152). When the entire input is a single text token of quote characters, this results in quadratic behavior.

PoC
const md = require('markdown-it');
const instance = md({ typographer: true });

// 160,000 consecutive double-quote characters
const payload = '"'.repeat(160000);

console.time('render');
instance.render(payload);
console.timeEnd('render');
// Output: render: ~21000ms (21 seconds)

// Compare with typographer disabled:
const safe = md({ typographer: false });
console.time('render-safe');
safe.render(payload);
console.timeEnd('render-safe');
// Output: render-safe: ~8ms

Measured timing on a modern system:

  • 10,000 quotes: ~19ms
  • 20,000 quotes: ~51ms
  • 40,000 quotes: ~212ms
  • 80,000 quotes: ~5,430ms
  • 160,000 quotes: ~21,198ms

The scaling is clearly superlinear (quadratic), with the 80K->160K step showing a ~3.9x increase for a 2x input increase, consistent with O(n^2).

Impact

Applications that render user-supplied markdown with typographer: true are vulnerable to denial of service. An attacker can submit a relatively small payload (160KB of quote characters) that causes the server to spend over 21 seconds processing a single request. Repeated submissions can exhaust server CPU resources and prevent legitimate users from being served.

The impact is mitigated by the fact that the typographer option defaults to false and must be explicitly enabled. However, the typographer feature is commonly enabled in production applications that want smart typography, and the markdown-it documentation prominently suggests enabling it.

A suggested fix would be to replace the replaceAt() approach with an array-based or StringBuilder-style approach that collects all replacements and applies them in a single pass, reducing the time complexity to O(n).

Severity

  • CVSS Score: 5.3 / 10 (Medium)
  • Vector String: CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:L

References

This data is provided by the GitHub Advisory Database (CC-BY 4.0).


markdown-it linkify: true has two quadratic paths, so a few hundred KB of markdown blocks the event loop for tens of seconds

GHSA-253c-mchw-3w2r

More information

Details

Summary

Two independent quadratic paths in the linkify: true handling. Both are in markdown-it's own code rather than in linkify-it, which stays linear on both payloads.

src/rules_core/linkify.ts calls arrayReplaceAt once per linkified text token, and that rebuilds the whole children array each time. A paragraph of N soft-broken lines is one inline token with about 2N children, so you get N rebuilds over a 2N array. Schema-less emails are what reach it. A http:// link gets consumed by the inline rule first and never arrives as a text token, so those stay linear.

src/rules_inline/linkify.ts runs state.pending.match(SCHEME_RE) at every :// in the source. state.pending only gets truncated once a link is actually produced, so an unregistered scheme leaves it growing and every :// rescans the lot.

Proof of concept

Clean install of 15.0.0 from npm, new MarkdownIt({ linkify: true }).render(payload), Node 24.

N 5,000 10,000 20,000 40,000
'a@b.co\n'.repeat(N), 34KB to 273KB 1.1s 4.7s 19.6s 89s
'a://'.repeat(N), 20KB to 156KB 0.25s 0.76s 3.1s 15.0s

Doubling the input roughly quadruples the time in both. With linkify: false the same inputs run in 25 to 131ms and stay flat.

Controls for the first one, all at N=20000: putting every email in a single text token ('a@b.co ') takes 0.59s, one email per paragraph ('a@b.co\n\n') takes 0.64s, and soft-broken lines with nothing linkifiable take 42ms. So it needs many children AND many of them linkifying. For the second, replaying just the SCHEME_RE calls against the same growing prefixes with no markdown-it involved accounts for 11.7s of the 15s.

Ordinary prose does it too. 'ping a@b.co ok\n'.repeat(20000) is 293KB and takes 31s.

Caveat

linkify is off by default, so this only reaches apps that turn it on.

Impact

Availability only. A few hundred KB of fairly ordinary markdown pins one core for tens of seconds, and because it's quadratic it degrades quickly with size. Nothing is read, written or executed.

Severity

  • CVSS Score: 6.3 / 10 (Medium)
  • Vector String: CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:N/VC:N/VI:N/VA:L/SC:N/SI:N/SA:N

References

This data is provided by the GitHub Advisory Database (CC-BY 4.0).


Release Notes

markdown-it/markdown-it (markdown-it)

v14.3.2

Compare Source

v14.3.1

Compare Source

v14.3.0

Compare Source

Changed
  • Reworked build pipeline & tools.
  • Added source maps.
  • Bumped linkify-it to 5.0.2.
Fixed
  • Preserve backslash-space hard line breaks, matching CommonMark 6.7, #​1185.

v14.2.0

Compare Source

Added
  • isPunctCharCode to utilities.
Fixed
  • Don't end HTML comment blocks on a blank line, #​1155.
  • Properly recognize astral chars (surrogates) in delimiter scans for
    emphasis-like markers, #​1072. Big thanks to @​tats-u for his global efforts
    with improving CJK support.
  • Preserve unicode whitespaces when trimm headings/paragraphs, #​1074.
  • More strict entities decode to avoid false positives ;, #​1096.
  • Restore block parser state on fail in lheading rule, #​1131.
Security
  • Fixed poor smartquotes perfomance on > 70k quotes in single block
  • Bumped linkify-it to 5.0.1 with fixed potential perfomance issues.

v14.1.1

Compare Source

Security
  • Fixed regression from v13 in linkify inline rule. Specific patterns could
    cause high CPU use. Thanks to @​ltduc147 for report.

Configuration

📅 Schedule: (in timezone America/New_York)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

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

@renovate renovate Bot added the dependencies Upgrade or downgrade of project dependencies. label Apr 9, 2026
@renovate
renovate Bot requested a review from a team April 9, 2026 00:34
@renovate
renovate Bot requested review from a team and sullivanpj as code owners April 9, 2026 00:34
@renovate

renovate Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor Author

Branch automerge failure

This PR was configured for branch automerge. However, this is not possible, so it has been raised as a PR instead.

@renovate
renovate Bot enabled auto-merge (squash) April 9, 2026 00:34
@renovate renovate Bot added the dependencies Upgrade or downgrade of project dependencies. label Apr 9, 2026
@deepsource-io

deepsource-io Bot commented Apr 9, 2026 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 242a5a8...5fd6414 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Oct 3, 2026 11:31p.m. Review ↗
Shell Oct 3, 2026 11:31p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] - autoclosed Apr 27, 2026
@renovate renovate Bot closed this Apr 27, 2026
auto-merge was automatically disabled April 27, 2026 17:55

Pull request was closed

@renovate
renovate Bot deleted the renovate/npm-markdown-it-vulnerability branch April 27, 2026 17:55
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] - autoclosed chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Apr 27, 2026
@renovate renovate Bot reopened this Apr 27, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch 3 times, most recently from fb1f18f to 0dabef2 Compare April 29, 2026 09:38
@renovate
renovate Bot enabled auto-merge (squash) April 29, 2026 09:38
@socket-security

socket-security Bot commented Apr 29, 2026 •

Copy link
Copy Markdown

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch 2 times, most recently from 955e7ec to 0770a6f Compare May 18, 2026 12:36
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.2.0 [security] May 28, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch 2 times, most recently from af398da to 6256de2 Compare May 28, 2026 23:08
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.2.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] May 28, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 6256de2 to 26e66aa Compare June 1, 2026 20:16
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.2.0 [security] Jun 1, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 26e66aa to 2251ab9 Compare June 2, 2026 01:57
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.2.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Jun 2, 2026
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.2.0 [security] Jun 11, 2026
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] Jul 11, 2026
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Jul 11, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from e9331f4 to 6a4a9f8 Compare July 11, 2026 23:16
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] Jul 18, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 6a4a9f8 to 57bb2ef Compare July 18, 2026 04:13
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Jul 18, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch 2 times, most recently from 8e66a96 to 0596a2c Compare July 20, 2026 21:08
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] Jul 20, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 0596a2c to 474203c Compare July 21, 2026 05:01
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Jul 21, 2026
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] Jul 24, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 474203c to 7ea7532 Compare July 24, 2026 22:11
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Jul 25, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch 2 times, most recently from 2462980 to 869c9ea Compare July 30, 2026 17:49
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] Jul 30, 2026
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Jul 30, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 869c9ea to 78928ee Compare July 30, 2026 23:13
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 78928ee to 183c57c Compare August 12, 2026 04:16
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] Aug 12, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 183c57c to 323fe14 Compare August 12, 2026 10:31
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Aug 12, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 323fe14 to 49d7527 Compare August 14, 2026 21:09
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] Aug 14, 2026
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.0 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] Aug 15, 2026
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 49d7527 to 6603879 Compare August 15, 2026 01:51
@renovate
renovate Bot force-pushed the renovate/npm-markdown-it-vulnerability branch from 6603879 to 5fd6414 Compare October 3, 2026 23:31
@renovate renovate Bot changed the title chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.1.1 [security] chore(monorepo): update pnpm-workspace.overrides markdown-it to >=14.3.2 [security] Oct 3, 2026

This branch has not been deployed

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

Labels

dependencies Upgrade or downgrade of project dependencies.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants