Skip to content

Commit 6828632

Browse files
committed
Adjust su-exec references, especially to note the severe years-long issue with 0.3
1 parent 2189d77 commit 6828632

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

INSTALL.md

-2
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@ RUN set -eux; \
5050

5151
## `FROM alpine` (3.7+)
5252

53-
**Note:** when using Alpine, it's probably also worth checking out [`su-exec`](https://github.com/ncopa/su-exec) (`apk add --no-cache su-exec`) instead, which since version 0.2 is fully `gosu`-compatible in a fraction of the file size.
54-
5553
```dockerfile
5654
ENV GOSU_VERSION 1.17
5755
RUN set -eux; \

README.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,6 @@ If you're curious about the edge cases that `gosu` handles, see [`Dockerfile.tes
5858

5959
## Alternatives
6060

61-
### `su-exec`
62-
63-
As mentioned in `INSTALL.md`, [`su-exec`](https://github.com/ncopa/su-exec) is a very minimal re-write of `gosu` in C, making for a much smaller binary, and is available in the `main` Alpine package repository.
64-
6561
### `chroot`
6662

6763
With the `--userspec` flag, `chroot` can provide similar benefits/behavior:
@@ -82,6 +78,10 @@ USER PID %CPU %MEM VSZ RSS TTY STAT START TIME COMMAND
8278
nobody 1 5.0 0.0 9592 1252 pts/0 RNs+ 23:21 0:00 ps faux
8379
```
8480

81+
### `su-exec`
82+
83+
In the Alpine Linux ecosystem, [`su-exec`](https://github.com/ncopa/su-exec) is a minimal re-write of `gosu` in C, making for a much smaller binary, and is available in the `main` Alpine package repository. However, as of version 0.3 it has [a pretty severe parser bug](https://github.com/ncopa/su-exec/pull/26) that hasn't been in a release for many years (and which the buggy behavior is that typos lead to running code as root unexpectedly 😬).
84+
8585
### Others
8686

8787
I'm not terribly familiar with them, but a few other alternatives I'm aware of include:

0 commit comments

Comments
 (0)