fix(provider): allow custom LLM HTTP hostnames (#929) - #935
Merged
H-Chris233 merged 2 commits intoAug 9, 2026
Conversation
H-Chris233
marked this pull request as ready for review
August 9, 2026 06:23
- 完全放开 http(s) 端点限制：公网/私网/本地/元数据地址一律放行， 校验仅保留 URL 格式检查（合法 http(s) URL + 主机名） - LLM / ASR / Omni 三条路径（连通性测试、模型列表、运行时）共用同一校验 - 运行时 resolve_ark_endpoint 补上同一校验，修复「保存端点绕过验证按钮检查」gap - 前端对 ark/asr/omni endpoint 的 http:// 输入展示明文风险提示（5 个 locale）
H-Chris233
force-pushed
the
codex/fix-929-http-domain-endpoint
branch
from
August 9, 2026 07:24
f53c287 to
d6896b7
Compare
Remove the provider-specific HTTPS-only guard so explicitly configured valid HTTP endpoints follow the shared endpoint validation policy.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
resolve_ark_endpoint补上同一校验，修复「保存端点绕过验证按钮检查」gap。ark/asr/omniendpoint 的http://输入展示明文风险提示（5 个 locale）。Validation
npm test— passed，43 frontend tests。cargo test --lib endpoint_security::tests— passed。cargo test --lib resolve_ark_endpoint -- --test-threads=1— passed。cargo test --lib asr_endpoint -- --test-threads=1— passed。