建议重构或修复关于天一弘短信服务
evil0th opened this issue · comments
Eric 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()));
有猫饼 commented
可以,来 PR 移除吧