Skip to content

fix: repair CTCC Cloud LVDN deployment - #1396

Merged
fudiwei merged 2 commits into
certimate-go:mainfrom
ickeep:agent/fix-ctcccloud-lvdn-registration
Jul 28, 2026
Merged

fix: repair CTCC Cloud LVDN deployment#1396
fudiwei merged 2 commits into
certimate-go:mainfrom
ickeep:agent/fix-ctcccloud-lvdn-registration

Conversation

@ickeep

@ickeep ickeep commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • fix the CTCC Cloud LVDN provider type mismatch (ctcccloud-ldvn vs. ctcccloud-lvdn)
  • route LVDN domain APIs through the documented cdnapi-global.ctapi.ctyun.cn endpoint
  • use the documented paths and HTTP methods for domain list, detail, and update requests
  • accept the current top-level code response format and normalize domain-list results for existing callers
  • correct the misleading cdn.QueryDomainList log label

Root cause

The UI persists ctcccloud-lvdn, but the backend registry used the misspelled ctcccloud-ldvn, so deployment failed before constructing the provider.

After fixing the registry key, the domain APIs still used the legacy ctlvdn-global endpoint and an outdated /v1/domain/query-domain-list route/response shape. The current CTCC Cloud API documents these requests under cdnapi-global.ctapi.ctyun.cn.

Validation

  • go test ./internal/domain ./internal/certmgmt/deployers ./pkg/sdk3rd/ctyun/lvdn
  • git diff --check
  • built a patched v0.4.28 linux/amd64 image and deployed it to an existing Certimate instance
  • ran a real LVDN workflow using certificate-SAN matching; certificate reuse, domain discovery, and certificate updates for three matched domains completed successfully

Fixes #1395

@ickeep
ickeep marked this pull request as ready for review July 26, 2026 06:57
Comment thread pkg/sdk3rd/ctyun/lvdn/client.go Outdated
Comment on lines +14 to +21
const endpoint = "https://ctlvdn-global.ctapi.ctyun.cn"
const (
endpoint = "https://ctlvdn-global.ctapi.ctyun.cn"
domainEndpoint = "https://cdnapi-global.ctapi.ctyun.cn"
)

type Client struct {
client *common.Client
client *common.Client
domainClient *common.Client

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

从修改结果上看,老的 endpoint 和 client 已经没有用了吧?为啥不直接替换掉呢?

@fudiwei fudiwei added bug Something isn't working pr: ready to merge and removed pr: waiting for review labels Jul 28, 2026
@fudiwei
fudiwei merged commit da60482 into certimate-go:main Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working pr: ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] 天翼云 LVDN 部署器未注册且域名 API 使用错误网关

2 participants