Skip to content

fix(openclaw-plugin): authenticate auto-recall health precheck#2384

Merged
qin-ctx merged 2 commits into
volcengine:mainfrom
MaojiaSheng:main
Jun 2, 2026
Merged

fix(openclaw-plugin): authenticate auto-recall health precheck#2384
qin-ctx merged 2 commits into
volcengine:mainfrom
MaojiaSheng:main

Conversation

@MaojiaSheng
Copy link
Copy Markdown
Collaborator

核心是把 auto-recall 的 health precheck 从“匿名请求 /health”改为“复用 OpenVikingClient.healthCheck()”,这样会带上已有的鉴权与租户头,避免在需要认证的网关前被误判失败。

关键位置:

• examples/openclaw-plugin/auto-recall.ts:202
• examples/openclaw-plugin/process-manager.ts:19
• examples/openclaw-plugin/client.ts:365
• examples/openclaw-plugin/tests/ut/context-engine-assemble.test.ts:138

验证情况

已跑定向单测,通过:

• examples/openclaw-plugin/tests/ut/client.test.ts
• examples/openclaw-plugin/tests/ut/context-engine-assemble.test.ts

Reuse the OpenViking client for auto-recall prechecks so remote /health probes carry the same auth and tenant headers as normal requests, preventing false recall skips behind authenticated gateways.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🏅 Score: 95
🧪 PR contains tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

PR Code Suggestions ✨

No code suggestions found for the PR.

@MaojiaSheng MaojiaSheng requested a review from Mijamind719 June 2, 2026 08:00
@qin-ctx qin-ctx merged commit 57b2e2a into volcengine:main Jun 2, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in OpenViking project Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants