Merge pull request #725 from petradonka/patch-1
Fix docker-setup.sh crash with optional env vars under set -u
This commit is contained in:
@@ -25,6 +25,7 @@
|
|||||||
- Gateway/WebChat: include handshake validation details in the WebSocket close reason for easier debugging.
|
- Gateway/WebChat: include handshake validation details in the WebSocket close reason for easier debugging.
|
||||||
- Gateway/Auth: send invalid connect responses before closing the handshake; stabilize invalid-connect auth test.
|
- Gateway/Auth: send invalid connect responses before closing the handshake; stabilize invalid-connect auth test.
|
||||||
- Doctor: surface plugin diagnostics in the report.
|
- Doctor: surface plugin diagnostics in the report.
|
||||||
|
- Docker: tolerate unset optional env vars in docker-setup.sh under strict mode. (#725) — thanks @petradonka.
|
||||||
- CLI/Update: preserve base environment when passing overrides to update subprocesses. (#713) — thanks @danielz1z.
|
- CLI/Update: preserve base environment when passing overrides to update subprocesses. (#713) — thanks @danielz1z.
|
||||||
- Agents: treat message tool errors as failures so fallback replies still send; require `to` + `message` for `action=send`. (#717) — thanks @theglove44.
|
- Agents: treat message tool errors as failures so fallback replies still send; require `to` + `message` for `action=send`. (#717) — thanks @theglove44.
|
||||||
- Agents: route subagent transcripts to the target agent sessions directory and add regression coverage. (#708) — thanks @xMikeMickelson.
|
- Agents: route subagent transcripts to the target agent sessions directory and add regression coverage. (#708) — thanks @xMikeMickelson.
|
||||||
|
|||||||
@@ -132,7 +132,7 @@ upsert_env() {
|
|||||||
local replaced=false
|
local replaced=false
|
||||||
for k in "${keys[@]}"; do
|
for k in "${keys[@]}"; do
|
||||||
if [[ "$key" == "$k" ]]; then
|
if [[ "$key" == "$k" ]]; then
|
||||||
printf '%s=%s\n' "$k" "${!k}" >>"$tmp"
|
printf '%s=%s\n' "$k" "${!k-}" >>"$tmp"
|
||||||
seen["$k"]=1
|
seen["$k"]=1
|
||||||
replaced=true
|
replaced=true
|
||||||
break
|
break
|
||||||
@@ -146,7 +146,7 @@ upsert_env() {
|
|||||||
|
|
||||||
for k in "${keys[@]}"; do
|
for k in "${keys[@]}"; do
|
||||||
if [[ -z "${seen[$k]:-}" ]]; then
|
if [[ -z "${seen[$k]:-}" ]]; then
|
||||||
printf '%s=%s\n' "$k" "${!k}" >>"$tmp"
|
printf '%s=%s\n' "$k" "${!k-}" >>"$tmp"
|
||||||
fi
|
fi
|
||||||
done
|
done
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user