Skip to content

WEBUI-2038: Remove stale tmp@0.2.3 from root lockfile (GHSA-52f5-9888-hmc6)#3173

Merged
madhurkulshrestha-hyland merged 1 commit into
lts-2025from
task-webui-2038-regen-root-lockfile-lts2025
May 21, 2026
Merged

WEBUI-2038: Remove stale tmp@0.2.3 from root lockfile (GHSA-52f5-9888-hmc6)#3173
madhurkulshrestha-hyland merged 1 commit into
lts-2025from
task-webui-2038-regen-root-lockfile-lts2025

Conversation

@madhurkulshrestha-hyland

Copy link
Copy Markdown
Contributor

Summary

Removes the stale packages/nuxeo-web-ui-ftest/node_modules/tmp entry from the root package-lock.json to eliminate the Dependabot alert for GHSA-52f5-9888-hmc6 (tmp arbitrary file/directory write via symlink).

Problem

The root lockfile had a nested entry pinning tmp@0.2.3 for the ftest workspace sub-tree. Despite the "tmp": "^0.2.4" override in both root and ftest package.json, npm was not updating this lockfile entry (npm limitation with file: linked packages).

Fix

Removed the 10-line stale entry entirely. With it gone, npm correctly hoists tmp@0.2.5 from root node_modules/ (via the override).

Verified stable: After removal, subsequent rm -rf node_modules && npm install cycles do NOT recreate the ftest sub-tree entry — npm continues to use the hoisted 0.2.5.

Result

$ npm audit | grep tmp → (no matches)
$ npm audit → 7 vulnerabilities (all from @open-wc/karma-esm, unfixable without migration)

Removed the packages/nuxeo-web-ui-ftest/node_modules/tmp entry that was
pinned to vulnerable 0.2.3. With this entry gone, npm correctly hoists
tmp@0.2.5 from root node_modules (via the override). The lockfile is
stable across npm install cycles — the entry does not reappear.

Eliminates Dependabot alert GHSA-52f5-9888-hmc6 (tmp symlink write).
@madhurkulshrestha-hyland madhurkulshrestha-hyland requested a review from a team as a code owner May 21, 2026 18:41
@madhurkulshrestha-hyland madhurkulshrestha-hyland requested review from AnilKumarVanga, Nishant0928 and Copilot and removed request for a team May 21, 2026 18:41

Copilot AI 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.

Copilot wasn't able to review any files in this pull request.

@sonarqubecloud

Copy link
Copy Markdown

@madhurkulshrestha-hyland madhurkulshrestha-hyland merged commit 50125e4 into lts-2025 May 21, 2026
13 of 14 checks passed
@madhurkulshrestha-hyland madhurkulshrestha-hyland deleted the task-webui-2038-regen-root-lockfile-lts2025 branch May 21, 2026 18:46
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