Sentinel icon indicating copy to clipboard operation
Sentinel copied to clipboard

[BUG] com.alibaba.csp.sentinel.transport.endpoint.EndPoint#toString()打印格式有误

Open guanweiqiang opened this issue 1 month ago • 5 comments

Issue Description

Type: bug report

Describe what happened

EndPoint的toString方法,打印host字段时格式有误 这是实际打印的:{Endpoint{protocol=HTTP, host='localhost, port=8900}

Describe what you expected to happen

预期打印的应该是:{Endpoint{protocol=HTTP, host=localhost, port=8900} 或者 {Endpoint{protocol=HTTP, host='localhost', port=8900}

How to reproduce it (as minimally and precisely as possible)

  1. 自己打印或者连接sentinel的dashboard,日志中会出现

Tell us your environment

sentinel1.8.9

Anything else we need to know?

guanweiqiang avatar Nov 12 '25 07:11 guanweiqiang

@LearningGp 我查看日志,发现报错来自项目中已有的 README 文件,我不确定是否是我修改的代码引起的。

Layau-code avatar Nov 25 '25 09:11 Layau-code

@LearningGp 你好,我把这次修复拆成了 3 个 PR(按时间顺序)并把当前进度写在下面,麻烦看一下是否 OK:

  1. PR-#3574: 初次提交 — 首次 PR(修复代码,但未添加单元测试),CI 报错 document-lint(MD060)因为 README 表格样式不满足规则,所以失败。
  2. PR-#3578: 继发提交 — 在 PR-A 基础上补充了测试(EndpointTest),验证修复生效,CI(单元测试)已通过,但 README 样式问题未改。
  3. PR-#3579: 文档提交 — 单独提交了 README 表格样式修复(MD060),并建议在仓库根 .gitignore 加入 node_modules/(避免意外提交大量依赖)。

说明/问题:

• 我不确定文档修改是否完全符合项目的文档风格,如果你们有其它规范(或 prefer 仅在 sentinel-dashboard 子目录里忽略 node_modules),请指出,我会按维护者意见更新 PR。 • .gitignore 中加入 node_modules/ 的原因是为了防止误提交本地依赖和减小仓库体积。若项目策略不允许全局忽略 node_modules(或某些模块需要保留 node_modules 被追踪),我可以撤回该行并只在 sentinel-dashboard 局部 .gitignore 保持该规则。

Layau-code avatar Nov 25 '25 16:11 Layau-code

了解,现有README被扫描出问题应该是由于规则变更导致的,感谢修复,我会先处理(#3579),另外两个PR建议合并哈

LearningGp avatar Dec 08 '25 08:12 LearningGp

可以review一下PR #3578,它CI测试超时了

Layau-code avatar Dec 19 '25 02:12 Layau-code

@LearningGp 好像只是CI超时了,代码应该没什么问题

Layau-code avatar Dec 21 '25 11:12 Layau-code