fix: portable vault path resolution across Windows/Mac/Linux
Replace hardcoded D:/vault references with candidate-list pattern that also checks $HOME/vault, ~/.vault, and respects VAULT_PATH env var override. Fixes vault.sh lookup failures on Mac and Howard's machine. Affected: CLAUDE.md, syncro.md, get-token.sh, patch-tenant-admin-manifest.sh Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -30,13 +30,21 @@ When invoked, use the Syncro REST API via `curl`. All requests include `?api_key
|
||||
### Get API key
|
||||
|
||||
```bash
|
||||
API_KEY=$(bash D:/vault/scripts/vault.sh get-field msp-tools/syncro.sops.yaml credentials.credential)
|
||||
# Portable vault resolver — works on Windows (D:/vault), Mac (~/.vault or ~/vault), Linux
|
||||
VAULT_SH=""
|
||||
for _c in "D:/vault/scripts/vault.sh" "$HOME/vault/scripts/vault.sh" "/d/vault/scripts/vault.sh" "$HOME/.vault/scripts/vault.sh"; do
|
||||
[[ -f "$_c" ]] && VAULT_SH="$_c" && break
|
||||
done
|
||||
[[ -z "$VAULT_SH" ]] && { echo "ERROR: vault.sh not found" >&2; exit 1; }
|
||||
|
||||
API_KEY=$(bash "$VAULT_SH" get-field msp-tools/syncro.sops.yaml credentials.credential)
|
||||
BASE="https://computerguru.syncromsp.com/api/v1"
|
||||
```
|
||||
|
||||
If `vault.sh get-field` fails (yq not installed), fall back to:
|
||||
```bash
|
||||
API_KEY=$(sops -d D:/vault/msp-tools/syncro.sops.yaml | py -c "import sys,yaml; print(yaml.safe_load(sys.stdin)['credentials']['credential'])")
|
||||
VAULT_ROOT=$(dirname "$(dirname "$VAULT_SH")")
|
||||
API_KEY=$(sops -d "$VAULT_ROOT/msp-tools/syncro.sops.yaml" | py -c "import sys,yaml; print(yaml.safe_load(sys.stdin)['credentials']['credential'])")
|
||||
```
|
||||
|
||||
### Endpoints reference
|
||||
|
||||
Reference in New Issue
Block a user