citahub / cita

A high performance blockchain kernel for enterprise users.
https://www.citahub.com/
Apache License 2.0
1.31k stars 210 forks source link

two improvments #931

Closed rink1969 closed 4 years ago

rink1969 commented 4 years ago

最近发现两类用户经常犯的小错误:

  1. 创建链的配置文件时,节点网络地址列表中不小心加入空格。创建脚本不会报错,运行的时候network报网络地址无法解析。
  2. setup/start节点的时候,比如 bin/cita setup test-chain/0 因为最后一个参数(节点名)本身就是目录,很容易后面多一个 /。 导致节点运行之后报错连接不上amqp。

这两个问题都很小,但是出问题之后定位还是要花一些时间的。 因此在代码中加入容错处理:

  1. 创建链配置时,对节点网络地址列表进行strip处理,去除多余的空格。
  2. 如果发现节点名是 / 结尾的,则自动将其去除。