Skip to content

fix(check-tool-updates): replace GNU-only tac and date -d <RFC> so socat lookup works on macOS #26

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

Merged
potiuk merged 1 commit into from
May 1, 2026
Merged

fix(check-tool-updates): replace GNU-only tac and date -d <RFC> so socat lookup works on macOS #26

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 25 additions & 5 deletions tools/agent-isolation/check-tool-updates.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,14 @@ for r in json.load(sys.stdin):

# socat upstream is a static HTML index; scrape the highest version
# tarball whose mtime is older than `cooldown_days`.
#
# Implementation note — portability. Both `tac` (used to reverse a
# sorted list) and `date -d <RFC-2822-string>` (used to parse the
# server's Last-Modified header) are GNU-only. macOS ships BSD
# coreutils which have neither. The version reversal is folded into
# `sort -uVr` (descending unique versions); date parsing is delegated
# to python3's `email.utils.parsedate_to_datetime`, which is the
# stdlib HTTP-date parser and is available everywhere python3 is.
socat_latest_aged() {
local cooldown_days="$1"
local cutoff=$(( now_epoch - cooldown_days * 86400 ))
Expand All @@ -100,19 +108,31 @@ socat_latest_aged() {
# `Last-Modified` (per HEAD) is older than the cutoff.
local index versions
index="$(curl -fsSL http://www.dest-unreach.org/socat/download/)" || return 1
versions=$(echo "$index" | grep -oE 'socat-[0-9]+\.[0-9]+\.[0-9]+\.[0-9]+\.tar\.gz' | sort -uV)
for v in $(echo "$versions" | tac); do
local ver mtime mtime_epoch
versions=$(echo "$index" | grep -oE 'socat-[0-9]+\.[0-9]+\.[0-9]+\.[0-9]+\.tar\.gz' | sort -uVr)
for v in $versions; do
local ver mtime parsed mtime_epoch mtime_date
ver="${v#socat-}"
ver="${ver%.tar.gz}"
mtime=$(curl -sI "http://www.dest-unreach.org/socat/download/${v}" \
| awk -F': ' '/^[Ll]ast-[Mm]odified:/ {print $2}' | tr -d '\r')
if [[ -z "$mtime" ]]; then
continue
fi
mtime_epoch=$(date -d "$mtime" +%s 2>/dev/null) || continue
parsed=$(MTIME="$mtime" python3 <<'PY' || true
import os
from email.utils import parsedate_to_datetime
try:
dt = parsedate_to_datetime(os.environ["MTIME"])
print(f"{int(dt.timestamp())}\t{dt.strftime('%Y-%m-%d')}")
except Exception:
raise SystemExit(1)
PY
)
[[ -z "$parsed" ]] && continue
mtime_epoch=${parsed%%$'\t'*}
mtime_date=${parsed##*$'\t'}
if (( mtime_epoch <= cutoff )); then
printf '%s\t%s\n' "$ver" "$(date -u -d "$mtime" +%Y-%m-%d)"
printf '%s\t%s\n' "$ver" "$mtime_date"
return 0
fi
done
Expand Down
Loading