Skip to content

Commit fcbe3e7

Browse files
authored
fix(prebuilt/alloydb-omni): require password env var explicitly (#3398)
## Description The `alloydb-omni` prebuilt config declared `password: ${ALLOYDB_OMNI_PASSWORD:}` (empty default), but the underlying `postgres` source marks `Password` as `validate:"required"`. An unset `ALLOYDB_OMNI_PASSWORD` therefore resolves to `""` → `null` and fails config validation anyway — the empty default was misleading. This drops the empty default and declares the var bare-required (`${ALLOYDB_OMNI_PASSWORD}`), matching the adjacent `user` field. Behavior is unchanged for the served path (an unset password already failed); it just makes the requirement honest and lets offline tooling substitute the placeholder. Stacked on #3388.
1 parent 6392824 commit fcbe3e7

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

‎internal/prebuiltconfigs/tools/alloydb-omni.yaml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ host: ${ALLOYDB_OMNI_HOST:localhost}
1919
port: ${ALLOYDB_OMNI_PORT:5432}
2020
database: ${ALLOYDB_OMNI_DATABASE}
2121
user: ${ALLOYDB_OMNI_USER}
22-
password: ${ALLOYDB_OMNI_PASSWORD:}
22+
password: ${ALLOYDB_OMNI_PASSWORD}
2323
queryParams: ${ALLOYDB_OMNI_QUERY_PARAMS:}
2424
---
2525
kind: tool

0 commit comments

Comments
 (0)