Skip to content

fix: reject empty clash proxy groups#378

Merged
7Sageer merged 1 commit into
mainfrom
codex/issue-277-empty-proxy-group-validation
Apr 26, 2026
Merged

fix: reject empty clash proxy groups#378
7Sageer merged 1 commit into
mainfrom
codex/issue-277-empty-proxy-group-validation

Conversation

@7Sageer

@7Sageer 7Sageer commented Apr 26, 2026

Copy link
Copy Markdown
Owner

Summary

Testing

  • npx vitest run

@vercel

vercel Bot commented Apr 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sublink-worker Building Building Preview Apr 26, 2026 0:11am

@7Sageer 7Sageer merged commit b51fa57 into main Apr 26, 2026
4 checks passed
@7Sageer 7Sageer deleted the codex/issue-277-empty-proxy-group-validation branch April 26, 2026 12:14
WayneWu98 added a commit to WayneWu98/sublink-worker that referenced this pull request May 7, 2026
Port from upstream 7Sageer/sublink-worker@b51fa570. validateProxyGroups now throws
InvalidConfigError when a url-test/fallback group has no proxies and
no providers, instead of silently filling with all available nodes.
The /clash endpoint returns HTTP 400 via existing handleError path.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
WayneWu98 added a commit to WayneWu98/sublink-worker that referenced this pull request May 7, 2026
Six fixes ported from upstream 7Sageer/sublink-worker (none of which
affect the Snell support added in v2.10.0):
- 7Sageer#371 custom rule proxy choices
- 7Sageer#382 xray subscription-userinfo
- subscription decoding (no spurious base64 decode of plain payloads)
- 7Sageer#378 reject empty Clash url-test/fallback groups
- 7Sageer#380 sing-box 1.11+ schema (no legacy block outbound, reject route action)
- 7Sageer#379 stable hash-based auto provider names

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant