Skip to content

update node-forge to latest version#1809

Merged
acosferreira merged 2 commits intomainfrom
ana/update-latest-node-forge
Dec 31, 2025
Merged

update node-forge to latest version#1809
acosferreira merged 2 commits intomainfrom
ana/update-latest-node-forge

Conversation

@acosferreira
Copy link
Contributor

@acosferreira acosferreira commented Dec 31, 2025

Jira Issue: https://issues.redhat.com/browse/AAP-61274

Assisted-by: @omaciel

Description

update package-lock to use latest version of node-forge (1.3.1 -> 1.3.4)

Testing

Steps to test

  1. Pull down the PR
  2. ...
  3. ...

Type of Change

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to break)
  • Security fix
  • Performance improvement
  • Code refactoring
  • Documentation update
  • CI/CD update

Backport Policy

This change should be:

  • Not backported - main/master only
  • Backported to specific releases (add labels after merge)

Automated Backport Instructions

After this PR is merged, add one or more labels to automatically create backport PRs:

  • backport/stable-2.4 - Backport to stable-2.4 branch
  • backport/stable-2.5 - Backport to stable-2.5 branch
  • backport/stable-2.6 - Backport to stable-2.6 branch
  • backport/all - Backport to all active stable branches
  • no-backport - Explicitly mark as not needing backport

Backport Justification

Special backport considerations:

Scenarios tested

Production deployment

  • This code change is ready for production on its own
  • This code change requires the following considerations before going to production:

Note

Updates dependency resolutions in package-lock.json for the Express stack.

  • Bumps express to 4.22.1, body-parser to 1.20.4, qs to 6.14.1, raw-body to 2.5.3, and @types/node-forge to 1.3.14
  • Loosens several sub-dependency ranges and updates transitive deps (e.g., http-errors@2.0.1, statuses@2.0.2), introducing side-channel-* and call-bound packages
  • Toggles some deps between devOptional and dev; no application code changes

Written by Cursor Bugbot for commit 447cd8d. This will update automatically on new commits. Configure here.

@acosferreira acosferreira enabled auto-merge (squash) December 31, 2025 18:19
@github-actions
Copy link

# npm audit report

qs  <6.14.1
Severity: high
qs's arrayLimit bypass in its bracket notation allows DoS via memory exhaustion - https://github.com/advisories/GHSA-6rw7-vpxm-498p
fix available via `npm audit fix`
node_modules/qs
  body-parser  <=1.20.3 || 2.0.0-beta.1 - 2.0.2
  Depends on vulnerable versions of qs
  node_modules/body-parser
    express  2.5.8 - 2.5.11 || 3.2.1 - 3.2.3 || 4.0.0-rc1 - 4.21.2 || 5.0.0-alpha.1 - 5.0.1
    Depends on vulnerable versions of body-parser
    Depends on vulnerable versions of qs
    node_modules/express

3 high severity vulnerabilities

To address all issues, run:
  npm audit fix

"version": "1.3.3",
"resolved": "https://registry.npmjs.org/node-forge/-/node-forge-1.3.3.tgz",
"integrity": "sha512-rLvcdSyRCyouf6jcOIPe/BgwG/d7hKjzMKOas33/pHEr6gbq18IK9zV7DiPvzsz0oBJPme6qr6H6kGZuI9/DZg==",
"devOptional": true,
Copy link

Choose a reason for hiding this comment

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

Security update not applied - version remains unchanged

The PR claims to be a security fix updating node-forge from version 1.3.1 to 1.3.4, but the actual node-forge package version remains at 1.3.3. The only changes are updating @types/node-forge (the TypeScript type definitions, not the actual library) and removing the devOptional flag. If a security vulnerability exists that requires version 1.3.4 or later, this PR does not actually address it. The security fix referenced in the PR description may not have been properly applied.

Fix in Cursor Fix in Web

@sonarqubecloud
Copy link

@acosferreira acosferreira merged commit 78c4275 into main Dec 31, 2025
13 checks passed
@acosferreira acosferreira deleted the ana/update-latest-node-forge branch December 31, 2025 18:37
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.

2 participants

Comments