Skip to content

feat(runtime): secure Nacos registration transport - #10

Merged
XnLemon merged 1 commit into
mainfrom
codex/provider-nacos-tls
Aug 10, 2026
Merged

feat(runtime): secure Nacos registration transport#10
XnLemon merged 1 commit into
mainfrom
codex/provider-nacos-tls

Conversation

@XnLemon

@XnLemon XnLemon commented Aug 10, 2026

Copy link
Copy Markdown
Member

Summary

  • make the Nacos API origin scheme explicitly select controlled plaintext or authenticated HTTPS registration
  • require a private CA and exact server name for HTTPS, with an optional complete mTLS client pair
  • share one fail-closed transport builder across Runtime A and Runtime B
  • reject system roots, proxies, redirects, downgrade, unsafe TLS files, and secret-bearing errors
  • document the deployment contract for both sample runtimes

Validation

  • go test -count=1 ./...
  • go test -race ./...
  • go vet ./...
  • real TLS and mTLS servers cover private-CA success, wrong CA, hostname mismatch, and required client authentication
  • bounded/non-regular/invalid material and redacted error tests

Ownership

This remains Samples deployment wiring. Core continues to receive an injected Nacos executor, and the consumer SDK does not gain registry ownership.

Fallback delta

Fallback delta: removed 1, retained 0, added 0, net -1

Removed: implicit system-root trust for HTTPS Provider registration.
Added fallback evidence: none

Closes #9

@XnLemon
XnLemon merged commit 1dcfae9 into main Aug 10, 2026
9 checks passed
@XnLemon
XnLemon deleted the codex/provider-nacos-tls branch August 10, 2026 12:03
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.

[Security] Require explicit TLS for Provider Nacos registration

1 participant