#17912: fix: configure git to use HTTPS instead of SSH for GitHub URLs
stale
size: XS
trusted-contributor
Cluster:
Update Handling Fixes
## Summary
When running `openclaw update` on a non-git install (global npm install), packages with git dependencies (like `whiskeysockets/libsignal-node`) were failing with "Permission denied (publickey)" error because npm tried to access them via SSH.
This fix configures git to use HTTPS instead of SSH for GitHub URLs during the global update step by setting the `GIT_CONFIG_PARAMETERS` environment variable.
## Changes
- Modified `src/infra/update-runner.ts` to pass git HTTPS configuration via environment variable when running global npm/pnpm/bun install
## Testing
- All existing tests in `src/infra/update-runner.test.ts` pass (13 tests)
- TypeScript compilation passes
- Code formatting verified
Fixes openclaw/openclaw#17907
<!-- greptile_comment -->
<h3>Greptile Summary</h3>
Fixes SSH authentication errors when installing packages with git dependencies during global `openclaw update` by configuring git to use HTTPS instead of SSH for GitHub URLs via the `GIT_CONFIG_PARAMETERS` environment variable.
<h3>Confidence Score: 5/5</h3>
- This PR is safe to merge with minimal risk
- The change is minimal, well-scoped, and addresses a specific SSH authentication issue. It only adds environment configuration for the global update step without modifying any other logic. All existing tests pass, and the solution follows standard git configuration practices.
- No files require special attention
<sub>Last reviewed commit: 863e350</sub>
<!-- greptile_other_comments_section -->
<!-- /greptile_comment -->
Most Similar PRs
#18961: fix: detect pnpm package manager in openclaw update
by norci · 2026-02-17
79.1%
#17237: fix(update): guard post-install imports after npm global update
by tdjackey · 2026-02-15
79.0%
#23139: test: fix flaky auth tests when OPENCLAW_GATEWAY_TOKEN is present
by Imccccc · 2026-02-22
78.5%
#10248: fix(update): default dev checkout path to ~/openclaw
by kimgyu · 2026-02-06
78.5%
#20585: fix: pre-flight ownership check before global update
by mwfj · 2026-02-19
78.1%
#17815: fix: use $HOME as cwd for global update to prevent path-dedot panic
by frankekn · 2026-02-16
77.8%
#8600: fix(update): add --ignore-scripts to prevent supply chain attacks
by yubrew · 2026-02-04
77.1%
#20330: Fix SSH tunnel startup on Windows by resolving ssh from PATH
by graysurf · 2026-02-18
77.0%
#6045: fix(auth): fix Gemini CLI path detection for global npm installs on...
by ehgamemo · 2026-02-01
76.1%
#18723: fix: use resolveEffectiveUpdateChannel to detect git tags for defau...
by MisterGuy420 · 2026-02-17
76.1%