Skip to content

Update axios dependency (fixes GHSA-43fc-jf86-j433)#55

Merged
knolleary merged 1 commit intonode-red:masterfrom
hlovdal:deps
Feb 11, 2026
Merged

Update axios dependency (fixes GHSA-43fc-jf86-j433)#55
knolleary merged 1 commit intonode-red:masterfrom
hlovdal:deps

Conversation

@hlovdal
Copy link
Contributor

@hlovdal hlovdal commented Feb 10, 2026

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Proposed changes

Updating this package fixes security issue GHSA-43fc-jf86-j433/CVE-2026-25639.

From my reading of https://github.com/axios/axios/blob/v1.x/CHANGELOG.md and https://github.com/axios/axios/releases the main new feature of version 1.13 is support for HTTP2 and no breaking changes, so stepping up should not represent a problem.

Checklist

  • I have read the contribution guidelines
  • For non-bugfix PRs, I have discussed this change on the forum/slack team.
  • I have run npm run test to verify the unit tests pass
  • I have added suitable unit tests to cover the new/changed functionality

$ npm audit

axios  <=1.13.4
Severity: high
Axios is Vulnerable to Denial of Service via __proto__ Key in mergeConfig - GHSA-43fc-jf86-j433
fix available via `npm audit fix --force`
Will install axios@1.13.5, which is outside the stated dependency range
node_modules/axios
hlovdal added a commit to hlovdal/hlovdal-node-red-lowercase-in-typescript that referenced this pull request Feb 10, 2026
Upstream issue, node-red/node-red-admin#55.

$ npm audit

axios  <=1.13.4
Severity: high
Axios is Vulnerable to Denial of Service via __proto__ Key in mergeConfig - GHSA-43fc-jf86-j433
fix available via `npm audit fix --force`
Will install node-red@4.0.9, which is a breaking change
node_modules/axios
  node-red-admin  0.2.0 - 3.1.0 || >=4.0.2
  Depends on vulnerable versions of axios
  node_modules/node-red-admin
    node-red  1.1.0-beta.1 - 2.2.3 || >=4.1.0-beta.1
    Depends on vulnerable versions of node-red-admin
    node_modules/node-red

3 high severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force
$ npm ls axios
@hlovdal/node-red-lowercase-in-typescript-cjs@1.0.0 /download/src/egne/hlovdal-node-red-lowercase-in-typescript
├─┬ node-red@4.1.4
│ └─┬ node-red-admin@4.1.2
│   └── axios@1.12.2
└─┬ wait-on@9.0.3
  └── axios@1.13.5
@coveralls
Copy link

Coverage Status

coverage: 81.27%. remained the same
when pulling ed232cd on hlovdal:deps
into 0209f78 on node-red:master.

@knolleary
Copy link
Member

Thank you! Was just coming to do this - so appreciate the PR.

@knolleary knolleary merged commit 4c4a762 into node-red:master Feb 11, 2026
6 checks passed
hlovdal added a commit to hlovdal/hlovdal-node-red-lowercase-in-typescript that referenced this pull request Feb 11, 2026
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.

3 participants