[PR #4172] [CLOSED] Update nodejs-common: enable npx, clarify #5052

New issue

Closed

opened 2026-05-05 10:31:27 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 10:31:27 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/netblue30/firejail/pull/4172
Author: @tredondo
Created: 4/8/2021
Status: ❌ Closed

Base: master ← Head: patch-8

📝 Commits (2)

6f89a87 Update nodejs-common: enable npx, clarify
26b0178 Forgot to remove the private-tmp line itself

📊 Changes

1 file changed (+8 additions, -7 deletions)

View changed files

📝 etc/profile-m-z/nodejs-common.profile (+8 -7)

📄 Description

Settings to enable npx (bundled with node).

Changed the order a bit to match the profile.template sections.

Removed the private-tmp comment because if an IDE debugs Node, then Node inherits the sandbox from the IDE. I wrote that mistaken PR early on when I didn't understand this aspect of firejail well.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/netblue30/firejail/pull/4172 **Author:** [@tredondo](https://github.com/tredondo) **Created:** 4/8/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `patch-8` --- ### 📝 Commits (2) - [`6f89a87`](https://github.com/netblue30/firejail/commit/6f89a877e0b01f56725b4771b71f496a63762f69) Update nodejs-common: enable npx, clarify - [`26b0178`](https://github.com/netblue30/firejail/commit/26b01789a475bbbb1e4844b0a8fae2ba7d28b1e5) Forgot to remove the private-tmp line itself ### 📊 Changes **1 file changed** (+8 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `etc/profile-m-z/nodejs-common.profile` (+8 -7) </details> ### 📄 Description Settings to enable npx ([bundled with node](https://nodejs.org/ca/blog/release/v8.2.0/)). Changed the order a bit to match the [profile.template sections](https://github.com/netblue30/firejail/blob/master/etc/templates/profile.template). Removed the private-tmp comment because if an IDE debugs Node, then Node inherits the sandbox from the IDE. I wrote [that mistaken PR](https://github.com/netblue30/firejail/pull/4085/files#diff-6d6766db31289911872e5c664b0e77be7ef045360c7341d53ff7567784866077R48) early on when I didn't understand this aspect of firejail well. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>