gsadmin/certctl
1
0
Fork 0
mirror of https://github.com/shankar0123/certctl.git synced 2026-06-07 14:21:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

docs(readme): fix quickstart $EDITOR portability bug

Browse Source 
The production-path quickstart at README.md:103-108 used `$EDITOR
deploy/.env` literally — assumes the operator has $EDITOR exported
in their shell. On a fresh macOS / zsh session (default install,
nothing in .zshrc), $EDITOR is unset and the shell expands the
command to ` deploy/.env` with a leading empty arg, which zsh tries
to execute as a binary:

  shankar@macbookpro certctl % $EDITOR deploy/.env
  zsh: permission denied: deploy/.env

The escalation reflex makes it worse — `sudo $EDITOR deploy/.env`
expands to `sudo deploy/.env` (sudo strips env by default), which
sudo dispatches as a command lookup against PATH:

  sudo: deploy/.env: command not found

Net: a new-user quickstart that fails on the second command of the
production path with two opaque errors back-to-back.

Replace with the POSIX-portable default-fallback form:

  "${EDITOR:-nano}" deploy/.env

`nano` is pre-installed on macOS (BSD nano) and every mainstream
Linux distro, so the fallback always resolves. The user's preferred
editor (vim/emacs/code) is still honored if they have $EDITOR set.
Added a parenthetical reminder so the operator who has a strong
editor preference knows they can substitute.

Verified no other phantom-EDITOR sites in README / docs/getting-started
/ docs/operator via:

  grep -nE '\$EDITOR\b' README.md docs/getting-started/*.md docs/operator/*.md
This commit is contained in:
shankar0123
2026-05-13 04:09:39 +00:00
parent 2ea5d5ec57
commit f390bd5142
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
README.md
+2 -1
View File
@@ -101,9 +101,10 @@ Wait ~30 seconds, then open **https://localhost:8443** in your browser. The demo
```bash
cp .env.example deploy/.env # or root .env if running outside compose
$EDITOR deploy/.env # set POSTGRES_PASSWORD, CERTCTL_AUTH_SECRET,
"${EDITOR:-nano}" deploy/.env # set POSTGRES_PASSWORD, CERTCTL_AUTH_SECRET,
# CERTCTL_API_KEY, CERTCTL_CONFIG_ENCRYPTION_KEY,
# CERTCTL_AGENT_ID — all via openssl rand
# (replace nano with your preferred editor)
docker compose -f deploy/docker-compose.yml up -d --build
```