Skip to content

docs(deps): correct the advisory count on the pypdf floor - #76

Merged
Prinevo merged 1 commit into
mainfrom
chore/pypdf-security-bump
Aug 27, 2026
Merged

docs(deps): correct the advisory count on the pypdf floor#76
Prinevo merged 1 commit into
mainfrom
chore/pypdf-security-bump

Conversation

@kmadan

@kmadan kmadan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Reduced to a one-line comment fix. Superseded in substance by #75.

When I opened this, pypdf was at >=6.14.2 with two open advisories. Dependabot's #75 merged 13 minutes later and raised the floor to 6.15.0, which clears both — so the constraint and lock changes I had here were redundant and I've dropped them. There are now zero open Dependabot alerts.

What remains is genuinely stale, though: #75 moved the floor but left the comment reading # 13 advisories, which was the count for the old 6.14.2 floor. It doesn't reflect the two that prompted the bump:

Advisory Issue
GHSA-fp3f-mc75-235c Large memory usage for large /ToUnicode streams
GHSA-fwg2-594c-jp42 Long runtimes / large memory for large CID font width ranges

That block exists specifically to record what each floor covers, so an inaccurate count in it is worth correcting.

-    "pypdf>=6.15.0",               # 13 advisories (multiple RAM-exhaust, infinite-loop fixes)
+    "pypdf>=6.15.0",               # 15 advisories (RAM-exhaust, infinite-loop, /ToUnicode and CID-width DoS)

Comment only. No constraint change, no lock change, no behaviour change. Close it instead if you'd rather not carry a docs-only PR.

@github-actions github-actions Bot added the 🚀 enhancement New feature or request label Aug 27, 2026
@kmadan
kmadan requested a review from Prinevo August 27, 2026 06:46

@Prinevo Prinevo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

approved

Dependabot's #75 raised the floor to 6.15.0 but left the comment reading
"13 advisories", which was the count for the previous 6.14.2 floor. The two
advisories that prompted the bump are not reflected:

  GHSA-fp3f-mc75-235c  large memory usage for large /ToUnicode streams
  GHSA-fwg2-594c-jp42  long runtimes / large memory for large CID font widths

That comment block exists to record what each floor actually covers, so a stale
count there is worth a line.

Comment only. No constraint or lock change: 6.15.0 already clears both, and
there are now zero open Dependabot alerts.
@kmadan
kmadan force-pushed the chore/pypdf-security-bump branch from 8ead378 to 2973be3 Compare August 27, 2026 06:49
@kmadan kmadan changed the title chore(deps): raise the pypdf floor to 6.15.0 to clear the last two Dependabot alerts docs(deps): correct the advisory count on the pypdf floor Aug 27, 2026
@Prinevo
Prinevo merged commit 0d99fb6 into main Aug 27, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants