fix(alias): validate endpoint URLs before saving

[overtrue]

Related Issue

Closes #181

Background

Issue #181 reports configuring an alias with http://rustfs-node{1...32}:9000, then seeing rc ls rustfs fail with Network error: dispatch failure.

That endpoint is RustFS server volume expansion syntax, not a single S3 service endpoint. rc alias set only rejected empty endpoints, so the invalid endpoint could be saved and fail later during the S3 request.

Solution

• Add shared alias endpoint validation in rc-core.
• Require HTTP(S) URLs with a host.
• Reject RustFS volume expansion patterns before saving aliases.
• Return a usage error from rc alias set when the endpoint is invalid.

Tests

• cargo test -p rc-core validate_alias_endpoint --lib
• cargo test -p rustfs-cli commands::alias::tests --lib
• RC_CONFIG_DIR=<tmp> cargo run -q -p rustfs-cli -- alias set rustfs 'http://rustfs-node{1...32}:9000' xxx xxx returns exit code 2
• make pre-commit

[Copilot]

Pull request overview

This PR introduces shared validation for alias endpoints so rc alias set rejects invalid S3 service URLs (notably RustFS volume expansion patterns) before persisting them, preventing later runtime/network failures.

Changes:

• Added validate_alias_endpoint to rc-core and re-exported it publicly.
• Reused shared HTTP(S)+host validation for env-based aliases and CLI input.
• Added unit tests in rc-core and CLI tests ensuring invalid endpoints are rejected with a usage error.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
crates/core/src/lib.rs	Re-exports the new shared endpoint validation API from rc-core.
crates/core/src/alias.rs	Adds validate_alias_endpoint, factors out HTTP(S)+host validation, and adds tests.
crates/cli/src/commands/alias.rs	Validates endpoints in alias set before saving; adds CLI test for invalid endpoint rejection.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.