NPM Resource Changed bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
13 Dec 2025

Pull Request for Issue # .

Summary of Changes

This pull request (PR) fixes three low severity security vulnerability in indirect NPM development dependencies reported by npm audit by using npm audit fix.

Testing Instructions

It needs a development environment with a git clone, composer and npm.

  1. If not done before, run composer install and npm ci.
  2. Run npm audit.
  3. Check the result.

Actual result BEFORE applying this Pull Request

# npm audit report

nodemailer  <=7.0.10
Nodemailer’s addressparser is vulnerable to DoS caused by recursive calls - https://github.com/advisories/GHSA-rcmh-qjqh-p98v
fix available via `npm audit fix`
node_modules/nodemailer
  mailparser  2.3.1 - 3.9.0
  Depends on vulnerable versions of nodemailer
  node_modules/mailparser
  smtp-server  2.0.0 - 3.16.1
  Depends on vulnerable versions of nodemailer
  node_modules/smtp-server

tinymce  <7.0.0
Severity: moderate
TinyMCE Cross-Site Scripting (XSS) vulnerability in handling external SVG files through Object or Embed elements - https://github.com/advisories/GHSA-5359-pvf2-pw78
fix available via `npm audit fix --force`
Will install tinymce@8.3.0, which is a breaking change
node_modules/tinymce

4 vulnerabilities (3 low, 1 moderate)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Expected result AFTER applying this Pull Request

# npm audit report

tinymce  <7.0.0
Severity: moderate
TinyMCE Cross-Site Scripting (XSS) vulnerability in handling external SVG files through Object or Embed elements - https://github.com/advisories/GHSA-5359-pvf2-pw78
fix available via `npm audit fix --force`
Will install tinymce@8.3.0, which is a breaking change
node_modules/tinymce

1 moderate severity vulnerability

To address all issues (including breaking changes), run:
  npm audit fix --force

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar richard67 richard67 - open - 13 Dec 2025
avatar richard67 richard67 - change - 13 Dec 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Dec 2025
Category NPM Change
avatar richard67 richard67 - change - 13 Dec 2025
Labels Added: NPM Resource Changed bug PR-5.4-dev
avatar brianteeman brianteeman - test_item - 13 Dec 2025 - Tested successfully
avatar brianteeman
brianteeman - comment - 13 Dec 2025

I have tested this item ✅ successfully on 359896f


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46571.

avatar richard67 richard67 - alter_testresult - 14 Dec 2025 - brianteeman: Tested successfully
avatar muhme muhme - test_item - 14 Dec 2025 - Tested successfully
avatar muhme
muhme - comment - 14 Dec 2025

I have tested this item ✅ successfully on 26e52f5

* Seen the 4 vulnerabilities (3 low, 1 moderate)

  • Applied PR with gh pr checkout 46571 and running npm audit report shows only the one moderate tinymce severity vulnerability remaining, as expected
  • Using node v24.11.1, saved package-lock.json file for comparisation, gone back with git switch -, did npm audit fix by own and got exactly the same package-lock.json file
  • NPM package versions are only updated in minor or patch versions

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46571.
avatar muhme muhme - change - 14 Dec 2025
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-12-14 14:41:23
Closed_By muhme
avatar muhme muhme - close - 14 Dec 2025
avatar muhme muhme - merge - 14 Dec 2025
avatar muhme
muhme - comment - 14 Dec 2025

Thank you @richard67 for your contribution. Thank you @brianteeman for your test.

Add a Comment

Login with GitHub to post a comment