sqle icon indicating copy to clipboard operation
sqle copied to clipboard

sqle 升级静态检查工具 golangci-lint 至 v1.53.2 版本

Open jessun opened this issue 2 years ago • 0 comments

存在问题

在当前的 main 分支(830352d)中,使用的 golangci-lint 版本为 v1.43.0。目前该版本下的检查未能发现部分类型断言panic的风险,比如 #1576。将该工具升级至 v1.53.2 后可以发现此类风险。

版本选择

根据 https://github.com/golangci/golangci-lint 的 tag 规律发现,该工具的末尾小版本号最大一般也就是到2,因此可以考虑使用最新版本 v1.53.2。

代码变更

  • 业务代码调整。在更新了 golangci-lint 工具后,发现有多处代码需要优化,没有业务变更。关联 pr 为 #1578。
  • 工具配置调整。在默认情况下发现较多问题时,在单次运行后 golangci-lint 不会显示所有问题,因此 .golangci.yml配置文件需要更新,以支持一次返回所有问题。

需要验证的场景

业务代码变更后,需要确保以下场景功能正常:

  • sqle 集成 cloudbeaver 相关功能使用;
  • sqle 登录功能;
  • dashboard 页面中「工作台」项目下拉列表;
  • MySQL 审核规则「索引创建建议」;
  • MySQL 审核「影响行数」功能;
  • 扫描任务 TiDB 审计日志;
  • 扫描任务 MySQL 慢日志(mysql slow log 文件);

jessun avatar Jun 14 '23 02:06 jessun