ballcat-projects / ballcat

😸一个快速开发脚手架,快速搭建企业级后台管理系统,并提供多种便捷starter进行功能扩展。主要功能包括前后台用户分离,菜单权限,数据权限,定时任务,访问日志,操作日志,异常日志,统一异常处理,XSS过滤,SQL防注入,国际化 等多种功能

Home Page:http://docs.ballcat.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

建议重构或修复关于天一弘短信服务

evil0th opened this issue · comments

commented

在上一个PR #260 时发现TianYiHongSenderImpl中一处bug,由于无法找到相关文档(现叫“不卡”,文档差异较大)去求证请求参数拼接逻辑,暂时按照原逻辑处理。

建议重构或者移除TianYiHong的实现。

错误逻辑处如下:占位符与参数个数不符合,按照这个逻辑根本不会把手机号码加入到参数中去。

String req = CharSequenceUtil.format("account={}&sign={}&datetime={}&content={}{}{}", 
	account.getUsername(),
	account.getPassword(), 
	md5.digestHex(account.getUsername() + account.getPassword() + dateTime),
	dateTime, 
	p.getContent()
	// sender id
	, ty.getSenderIdCountry().contains(p.getCountry()) ? "&senderid=" + ty.getSenderId() : ""
	// numbers
	, CharSequenceUtil.join(",", p.getPhoneNumbers().toArray()));

可以,来 PR 移除吧