Skip to content

fix: use displayServerUrl for worker download commands and improve admin UI#237

Merged
Thomas-Eliot merged 1 commit intohigress-group:mainfrom
lexburner:fix/display-server-url-improvements
Apr 1, 2026
Merged

fix: use displayServerUrl for worker download commands and improve admin UI#237
Thomas-Eliot merged 1 commit intohigress-group:mainfrom
lexburner:fix/display-server-url-improvements

Conversation

@lexburner
Copy link
Copy Markdown
Contributor

📝 Description

Worker 下载命令中优先使用 Nacos 的 displayServerUrl(公网展示地址),当未配置时回退到 serverUrl,解决内网地址场景下开发者无法直接使用下载命令的问题。同时改进管理后台 Nacos 配置页面的展示和表单提示。

  • WorkerServiceImpl:构建下载 URL 时优先取 displayServerUrl,未配置则回退到 serverUrl
  • NacosConsoles.tsx
    • 列表新增「展示地址」列,方便管理员直观查看
    • 表单中 displayServerUrl 输入框增加用途说明(extra 提示文字)和更具体的 placeholder 示例

🔗 Related Issues

✅ Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring (no functional changes)
  • Performance improvement
  • Build/CI configuration change
  • Other (please describe):

🧪 Testing

  • Manual testing completed
  • All tests pass locally

📋 Checklist

  • Code has been formatted (mvn spotless:apply for backend, npm run lint:fix for frontend)
  • Code is self-reviewed
  • No breaking changes (or migration guide provided)

…min UI

Worker 下载命令中优先使用 Nacos 的 displayServerUrl(公网展示地址),
当未配置时回退到 serverUrl,解决内网地址场景下开发者无法直接使用下载
命令的问题。同时改进管理后台 Nacos 配置页面的展示和表单提示。

- WorkerServiceImpl: 构建下载 URL 时优先取 displayServerUrl
- NacosConsoles.tsx: 列表新增「展示地址」列,表单增加用途说明

🤖 Generated with [Qoder][https://qoder.com]
Copy link
Copy Markdown
Collaborator

@Thomas-Eliot Thomas-Eliot left a comment

Choose a reason for hiding this comment

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

LGTM

@Thomas-Eliot Thomas-Eliot merged commit f1dc937 into higress-group:main Apr 1, 2026
17 checks passed
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.

2 participants