| ▲ | inbx0 5 days ago |
| Periodic reminder to disable npm install scripts. npm config set ignore-scripts true [--global]
It's easy to do both at project level and globally, and these days there are quite few legit packages that don't work without them. For those that don't, you can create a separate installation script to your project that cds into that folder and runs their install-script.I know this isn't a silver bullet solution to supply chain attakcs, but, so far it has been effective against many attacks through npm. https://docs.npmjs.com/cli/v8/commands/npm-config |
|
| ▲ | homebrewer 5 days ago | parent | next [-] |
| I also use bubblewrap to isolate npm/pnpm/yarn (and everything started by them) from the rest of the system. Let's say all your source code resides in ~/code; put this somewhere in the beginning of your $PATH and name it `npm`; create symlinks/hardlinks to it for other package managers: #!/usr/bin/bash
bin=$(basename "$0")
exec bwrap \
--bind ~/.cache/nodejs ~/.cache \
--bind ~/code ~/code \
--dev /dev \
--die-with-parent \
--disable-userns \
--new-session \
--proc /proc \
--ro-bind /etc/ca-certificates /etc/ca-certificates \
--ro-bind /etc/resolv.conf /etc/resolv.conf \
--ro-bind /etc/ssl /etc/ssl \
--ro-bind /usr /usr \
--setenv PATH /usr/bin \
--share-net \
--symlink /tmp /var/tmp \
--symlink /usr/bin /bin \
--symlink /usr/bin /sbin \
--symlink /usr/lib /lib \
--symlink /usr/lib /lib64 \
--tmpfs /tmp \
--unshare-all \
--unshare-user \
"/usr/bin/$bin" "$@"
The package manager started through this script won't have access to anything but ~/code + read-only access to system libraries: bash-5.3$ ls -a ~
. .. .cache code
bubblewrap is quite well tested and reliable, it's used by Steam and (IIRC) flatpak. |
| |
| ▲ | aorth 14 hours ago | parent | next [-] | | Very cool idea. Thanks for sharing. I made some minor tweaks based on feedback to your comment: #!/usr/bin/env bash
#
# See: https://news.ycombinator.com/item?id=45034496
bin=$(basename "$0")
echo "==========================="
echo "Wrapping $bin in bubblewrap"
echo "==========================="
exec bwrap \
--bind ~/.cache ~/.cache \
--bind "${PWD}" "${PWD}" \
--dev /dev \
--die-with-parent \
--disable-userns \
--new-session \
--proc /proc \
--ro-bind /etc/ca-certificates /etc/ca-certificates \
--ro-bind /etc/resolv.conf /etc/resolv.conf \
--ro-bind /etc/ssl /etc/ssl \
--ro-bind /usr /usr \
--setenv PATH /usr/bin \
--symlink /usr/bin /bin \
--symlink /usr/bin /sbin \
--symlink /usr/lib /lib \
--symlink /usr/lib64 /lib64 \
--tmpfs /tmp \
--unshare-all \
--unshare-user \
--share-net \
/usr/bin/env "$bin" "$@"
Notably `--share-net` should be moved down since it is negated by `--unshare-all`. I also added a reminder that the command is being bubblewrapped, modified the second read-write bind to the current directory, and changed the final exec to use `/usr/bin/env` to find the binary so it can be more flexible. I tested it with npm and yarn just now and it seems to work well. Thanks! | |
| ▲ | internet_points 5 days ago | parent | prev | next [-] | | Thanks, handy wrapper :) Note: --symlink /usr/lib /lib64 \
should probably be `/usr/lib64`and --share-net \
should go after the `--unshare-all --unshare-user`Also, my system doesn't have a symlink from /tmp to /var/tmp, so I'm guessing that's not needed for me (while /bin etc. are symlinks) | |
| ▲ | TheTaytay 5 days ago | parent | prev | next [-] | | Very cool. Hadn't heard of this before. I appreciate you posting it. | |
| ▲ | oulipo2 5 days ago | parent | prev | next [-] | | Will this work on osX? and for pnpm? | | |
| ▲ | aragilar 5 days ago | parent [-] | | No, bubblewrap uses linux namespaces. You can use for (almost) whatever software you want. |
| |
| ▲ | johnisgood 5 days ago | parent | prev | next [-] | | Firejail is quite good, too. I have been using firejail more than bubblewrap. | |
| ▲ | shermantanktop 5 days ago | parent | prev | next [-] | | This is trading one distribution problem (npx) for another (bubblewrap). I think it’s a reasonable trade, but there’s no free lunch. | | |
| ▲ | homebrewer 5 days ago | parent [-] | | Not sure what this means. bubblewrap is as free as it gets, it's just a thin wrapper around the same kernel mechanisms used for containers, except that it uses your existing filesystems instead of creating a separate "chroot" from an OCI image (or something like it). The only thing it does is hiding most of your system from the stuff that runs under it, whitelisting specific paths, and optionally making them readonly. It can be used to run npx, or anything else really — just shove move symblinks into the beginning of your $PATH, each referencing the script above. Run any of them and it's automatically restricted from accessing e.g. your ~/.ssh https://wiki.archlinux.org/title/Bubblewrap | | |
| ▲ | conception 5 days ago | parent [-] | | It means that someone just has to compromise bubblewrap instead of the other vectors. | | |
| ▲ | johnisgood 5 days ago | parent | next [-] | | This is such a defeatist perspective. You could say this about anything ad nauseum. I think bubblewrap (or firejail) is less likely to be a successful target. | |
| ▲ | haswell 5 days ago | parent | prev | next [-] | | While this may be true, this is still a major improvement, no? i.e. it seems far more likely that a rapidly evolving hot new project will be targeted vs. something more stable and explicitly security focused like bubblewrap. | |
| ▲ | throwawaysoxjje 5 days ago | parent | prev | next [-] | | Am I getting bubblewrap somewhere other than my distro? What makes it different from any other executable that comes from there? | | |
| ▲ | shermantanktop 4 days ago | parent [-] | | Nothing. Does your threat model assume 100% trust in your distro? I understand saying you trust it a lot more than the garbage on npm. But if your trust is anything less than 100%, you are balancing risk and benefit. |
| |
| ▲ | cozzyd 5 days ago | parent | prev | next [-] | | sure but surely one gets bubblewrap from their distro, and you have to trust your distro anyway. | |
| ▲ | theamk 5 days ago | parent | prev [-] | | Not "instead", it's "in addition to". Your classical defense-in-depth. | | |
| ▲ | oulipo2 5 days ago | parent [-] | | No, "instead". If they compromise bubblewrap to send out your files, and you run bubblewrap anyway for any reason, you're still compromised. But obviously you can probably safely pin bubblewrap to a given version, and you don't need to "install packages through it", which is the main weakness of package managers | | |
| ▲ | ChocolateGod 5 days ago | parent | next [-] | | Bubblewrap uses the same Linux functions that billion dollar cloud infrastructure use. Bubblewrap does no sandboxing/restrictions itself, it's instructing the kernel to do it. | |
| ▲ | aragilar 5 days ago | parent | prev [-] | | How? bubblewrap isn't something someone has randomly uploaded to npm, it has well known maintainers and a well organised release process (including package signing). Which is easier to do: upload a package to npm and get people to use it, or spend 2+ years trying to become a maintainer of bubblewrap or one of its dependencies to compromise it. | | |
| ▲ | oulipo2 5 days ago | parent [-] | | Sure, but there's plenty of packages with well-known maintainers who get compromised... | | |
| ▲ | haswell 5 days ago | parent [-] | | The fact that something can happen is separate from how likely that thing is to happen, and that’s what matters here. The comments here that point to this theoretical possibility seem to be missing the point, which is that using something like bubblewrap is an improvement over running arbitrary projects un-sandboxed, and the likelihood of such an attack is far less than the likelihood of any one of hundreds of rapidly evolving, lesser known, lesser scrutinized projects getting compromised. |
|
|
|
|
|
|
| |
| ▲ | 4 days ago | parent | prev [-] | | [deleted] |
|
|
| ▲ | tiagod 5 days ago | parent | prev | next [-] |
| Or use pnpm. The latest versions have all dependency lifecycle scripts ignored by default. You must whitelist each package. |
| |
| ▲ | chrisweekly 5 days ago | parent | next [-] | | pnpm is not only more secure, it's also faster, more efficient wrt disk usage, and more deterministic by design. | | |
| ▲ | norskeld 5 days ago | parent [-] | | It also has catalogs feature for defining versions or version ranges as reusable constants that you can reference in workspace packages. It was almost the only reason (besides speed) I switched a year ago from npm and never looked back. | | |
| ▲ | mirekrusin 5 days ago | parent [-] | | workspace protocol in monorepo is also great, we're using it a lot. | | |
| ▲ | dvfjsdhgfv 5 days ago | parent [-] | | OK so it seems too good now, what are the downsides? | | |
| ▲ | nawgz 5 days ago | parent | next [-] | | ‘pnpm’ is great, swapped to it a year ago after yarn 1->4 looked like a new project every version and npm had an insane dependency resolution issue for platform specific packages pnpm had good docs and was easy to put in place. Recommend | |
| ▲ | c-hendricks 5 days ago | parent | prev | next [-] | | If you relied on hoisting of transitive dependencies, you'll now have to declare that fact in a project's .npmrc Small price to pay for all the advantages already listed. | | | |
| ▲ | mirekrusin 5 days ago | parent | prev | next [-] | | Downside is that you have to add "p" in front, ie. instead of "npm" you have to type "pnpm". That's all that I'm aware of. | |
| ▲ | Nathanba 5 days ago | parent | prev | next [-] | | A few years ago it didn't work in all cases when npm did. It made me stop using it because I didn't want to constantly check with two tools. The speed boost is nice but I don't need to npm install that often. | |
| ▲ | TheRoque 5 days ago | parent | prev [-] | | Personally, I didn't find a way to create one docker image for each of my project (in a pnpm monorepo) in an efficient way | | |
|
|
|
| |
| ▲ | trw55 5 days ago | parent | prev | next [-] | | Same for bun, which I find faster than pnpm | |
| ▲ | jim201 5 days ago | parent | prev [-] | | This is the way. It’s a pain to manually disable the checks, but certainly better than becoming victim to an attack like this. |
|
|
| ▲ | ashishb 5 days ago | parent | prev | next [-] |
| I run all npm based tools inside Docker with no access beyond the current directory. https://ashishb.net/programming/run-tools-inside-docker/ It does reduce the attach surface drastically. |
|
| ▲ | dns_snek 5 days ago | parent | prev | next [-] |
| Whenever I read this well-meaning advice I have to ask: Do you actually read hundreds of thousands of lines of code (or more) that NPM installed? Because the workflow for 99.99% of developers is something resembling: 1. git clone 2. npm install (which pulls in a malicious dependency but disabling post-install scripts saved you for now!) 3. npm run (executing your malicious dependency, you're now infected) The only way this advice helps you is if you also insert "audit the entirety of node_modules" in between steps 2 and 3 which nobody does. |
| |
| ▲ | IshKebab 4 days ago | parent [-] | | Yeah I guess it probably helps you specifically, because most malware is going to do the lazy thing and use install scripts. But it doesn't help everyone in general because if e.g. NPM disabled those scripts entirely (or made them opt-in) then the malware authors would just put their malware into the `npm run` as you say. | | |
| ▲ | dns_snek 4 days ago | parent [-] | | Indeed it may save you in case the malware is being particularly lazy but I think it may do more harm than good by giving people a false sense of security and it can also break packages that use post-install scrips for legitimate reasons. For anyone who actually cares about supply chain attacks, the minimum you should be doing is running untrusted code in some sort of a sandbox that doesn't have access to important credentials like SSH keys, like a dev container of some sort. You would still need to audit the code otherwise you might ship a backdoor to production but it would at least protect you against a developer machine compromise... unless you get particularly unlucky and it also leverages a container escape 0-day, but that's secure enough for me personally. |
|
|
|
| ▲ | eitau_1 5 days ago | parent | prev | next [-] |
| Why the same advice doesn't apply to `setup.py` or `build.rs`? Is it because npm is (ab)used for software distribution (eg. see sibling comment: https://news.ycombinator.com/item?id=45041292) instead of being used only for managing library-dependencies? |
| |
| ▲ | ivape 5 days ago | parent | next [-] | | It should apply for anything. Truth be told the process of learning programming is so arduous at times that you basically just copy and paste and run fucking anything in terminal to get a project setup or fixed. Go down the rabbit hole of just installing LLM software and you’ll find yourself in quite a copy and paste frenzy. We got used to this GitHub shit of setting up every process of an install script in this way, so I’m surprised it’s not happening constantly. | |
| ▲ | username223 5 days ago | parent | prev | next [-] | | It should, and also to Makefile.PL, etc. These systems were created at a time when you were dealing with a handful of dependencies, and software development was a friendlier place. Now you're dealing with hundreds of recursive dependencies, all of which you should assume may become hostile at any time. If you neither audit your dependencies, nor have the ability to sue them for damages, you're in a precarious position. | |
| ▲ | ifwinterco 5 days ago | parent | prev [-] | | For simple python libraries setup.py has been discouraged for a long time in favour of pyproject.toml for exactly this reason |
|
|
| ▲ | halflife 5 days ago | parent | prev | next [-] |
| This sucks for libraries that download native binaries in their install script. There are quite a few. |
| |
| ▲ | lrvick 5 days ago | parent | next [-] | | Downloading binaries as part of an installation of a scripting language library should always be assumed to be malicious. Everything must be provided as source code and any compilation must happen locally. | | |
| ▲ | oulipo2 5 days ago | parent [-] | | Sure, but then you need to have a way to whitelist | | |
| ▲ | lrvick 5 days ago | parent [-] | | The whitelist is the package-lock.json of the hashes of libraries you or a security reviewer you trust has reviewed. |
|
| |
| ▲ | junon 5 days ago | parent | prev [-] | | You can still whitelist them, though, and reinstall them. |
|
|
| ▲ | andix 5 days ago | parent | prev | next [-] |
| I guess this won't help with something like nx. It's a CLI tool that is supposed to be executed inside the source code repo, in CI jobs or on developer pcs. |
| |
| ▲ | inbx0 5 days ago | parent [-] | | According to the description in advisory, this attack was in a postinstall script. So it would've helped in this case with nx. Even if you ran the tool, this particular attack wouldn't have been triggered if you had install scripts ignored. |
|
|
| ▲ | johnisgood 5 days ago | parent | prev | next [-] |
| At this point why not just avoid npm (and friends) like the plague? Genuinely curious. |
| |
| ▲ | ifwinterco 5 days ago | parent [-] | | I work for a company that needs to ship software so my salary can get paid | | |
| ▲ | johnisgood 5 days ago | parent [-] | | Can't you guys replace the most vulnerable parts with something better? I have been experimenting with Go + Fyne, it is pretty neat, all things considered. | | |
|
|
|
| ▲ | peacebeard 5 days ago | parent | prev | next [-] |
| Looks like pnpm 10 does not run lifecycle scripts of dependencies unless they are listed in ‘onlyBuiltDependencies’. Source: https://pnpm.io/settings#ignoredepscripts |
|
| ▲ | 5 days ago | parent | prev | next [-] |
| [deleted] |
|
| ▲ | no_wizard 5 days ago | parent | prev | next [-] |
| Pnpm natively lets you selectively enable it on a package basis |
|
| ▲ | arminiusreturns 5 days ago | parent | prev | next [-] |
| As a linux admin, I refuse to install npm or anything that requires it as a dep. It's been bad since the start. At least some people are starting to see it. |
| |
| ▲ | azangru 5 days ago | parent [-] | | > As a linux admin, I refuse to install npm or anything that requires it as a dep. It's been bad since the start. As a front-end web developer, I need a node package manager; and npm comes bundled with node. |
|
|
| ▲ | antihero 5 days ago | parent | prev | next [-] |
| I wonder how many other packages are going to be compromised due to this also. Like a network effect. |
|
| ▲ | sheerun 5 days ago | parent | prev | next [-] |
| Secondary reminder that it means nothing as soon as you run any of scripts or binaries |
|
| ▲ | herpdyderp 5 days ago | parent | prev | next [-] |
| Unfortunately this also blocks your own life cycle scripts. |
|
| ▲ | oulipo2 5 days ago | parent | prev | next [-] |
| Does it work the same for pnpm ? |
|
| ▲ | sieabahlpark 5 days ago | parent | prev [-] |
| [dead] |