Cache IP address for install output #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Draft

SunnkerLocket89 wants to merge 1 commit into master from codex/replace-grep-p-with-portable-alternative2026-01-2111-35-09

Owner

SunnkerLocket89 commented Jan 21, 2026

Motivation

Cache the resolved IPv4 address once to avoid repeating the ip ... | awk ... command substitution in the final webinterface messages and to improve readability.

Description

In install.sh compute ip_address=$(ip -o -4 route get 1.2.3.4 | awk '{for (i=1;i<=NF;i++) if ($i=="src"){print $(i+1); exit}}') and reuse http://${ip_address} in the lighttpd and nginx completion messages instead of duplicating the command substitution.

Testing

Ran bash -n install.sh to validate the script syntax and it succeeded.


          Cache IP address for install output

cda4a1c

SunnkerLocket89 added the codex label

— with

ChatGPT Codex Connector

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels