Skip to content

Override minimatch to 11 to close the glob amplification DoS [skip deploy] #no-deploy - #334

Merged
SebastianLopezO merged 1 commit into
mainfrom
security/dependabot/minimatch-globparts-dos
Nov 1, 2026
Merged

SebastianLopezO merged 1 commit into
mainfrom
security/dependabot/minimatch-globparts-dos

Conversation

@SebastianLopezO

Copy link
Copy Markdown
Member

Caso

  • Origen: Dependabot alerts, #131
    (GHSA-g3hv-75wg-m249, minimatch has potential denial of service via globparts amplification).
  • Severidad: media; afecta a minimatch < 11.0.1.
  • Análisis: minimatch 10.2.5 llega solo por ESLint 10 (@eslint/config-array y @typescript-eslint/typescript-estree),
    es dependencia de desarrollo y procesa los patrones de la configuración del propio repositorio. ESLint pide ^10,
    así que Dependabot no puede proponer la actualización.

Cambio

  • overrides: minimatch: ^11.0.1 en pnpm-workspace.yaml (lockfile con minimatch 11.0.3), con la nota de quitarlo
    cuando ESLint dependa de la 11.

Verificación

  • pnpm why minimatch → 11.0.3.
  • pnpm lint (ESLint usa minimatch para los patrones de la configuración), pnpm test y pnpm build en verde;
    pnpm install --frozen-lockfile pasa.

@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
npm/@isaacs/nonrecursive-glob-match 1.0.3 UnknownUnknown
npm/minimatch 11.0.3 🟢 5.5
Details
CheckScoreReason
Code-Review⚠️ 0Found 1/28 approved changesets -- score normalized to 0
Packaging⚠️ -1packaging workflow not detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Maintained🟢 42 commit(s) and 3 issue activity found in the last 90 days -- score normalized to 4
Binary-Artifacts🟢 10no binaries found in the repo
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
Security-Policy🟢 10security policy file detected
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0

Scanned Files

  • pnpm-lock.yaml

@SebastianLopezO
SebastianLopezO merged commit f78bbec into main Nov 1, 2026
8 of 9 checks passed
@SebastianLopezO
SebastianLopezO deleted the security/dependabot/minimatch-globparts-dos branch November 1, 2026 03:44
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