Skip to content

fix(openclaw-plugin): harden local server lifecycle#1033

Open
mobilebarn wants to merge 1 commit intovolcengine:mainfrom
mobilebarn:fix/local-server-lifecycle
Open

fix(openclaw-plugin): harden local server lifecycle#1033
mobilebarn wants to merge 1 commit intovolcengine:mainfrom
mobilebarn:fix/local-server-lifecycle

Conversation

@mobilebarn
Copy link
Copy Markdown

Summary

  • SIGTERM before SIGKILL: killProcessOnPortUnix now sends SIGTERM first, waits 1 second for graceful shutdown, then only escalates to SIGKILL if the process is still alive. Fixes hard-kill without cleanup.
  • Configurable auto-recall timeout: The hardcoded 5-second AUTO_RECALL_TIMEOUT_MS is now exposed as autoRecallTimeoutMs in plugin config (clamped 1000–30000ms), with defaults, validation, schema, and UI hints.
  • Log stripped proxy vars: When proxy environment variables (ALL_PROXY, HTTP_PROXY, HTTPS_PROXY and lowercase variants) are removed from the local server environment, they are now logged so users can diagnose connectivity issues.

Closes #964

- Use SIGTERM before SIGKILL for graceful shutdown of stale processes
- Make auto-recall timeout configurable via autoRecallTimeoutMs
- Log when proxy environment variables are stripped from local server

Closes volcengine#964
@github-actions
Copy link
Copy Markdown

Failed to generate code suggestions for PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

[Bug]: gateway can't start because of openviking failure: Error: OpenViking health check timeout at http://127.0.0.1:1933

1 participant