Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

消息验重问题 #333

Closed
zhouwei517 opened this issue Sep 12, 2017 · 4 comments
Closed

消息验重问题 #333

zhouwei517 opened this issue Sep 12, 2017 · 4 comments
Assignees
Labels

Comments

@zhouwei517
Copy link

WxMpMessageRouter的isMsgDuplicated方法。使用msgId来验重是不正确的。这个msgId可能重复,会造成部分人员收不到回信(我们在测试的时候15个人一块语音,这个ID重复了,15个人一块喊语音,有5-6人收不到)。暂时的解决方案是加了FromUser和CreateTime。请官方关注一下。另外WxCpMessageRouter可能会有同样的问题。最后,感谢开源。

@binarywang
Copy link
Owner

看来真是个问题,需要修复下,感谢及时提出此问题

@imtiaotiao
Copy link

这个bug修复了吗?我看2.8.0里面用的还是msgId

@binarywang binarywang self-assigned this Nov 7, 2017
@binarywang binarywang added the bug label Nov 7, 2017
@binarywang
Copy link
Owner

确实还没修复,因为这个问题是在2.8.0发布之后发现的

@binarywang
Copy link
Owner

测试版本2.8.7.BETA已修复

comeonc added a commit to comeonc/weixin-java-tools that referenced this issue Oct 9, 2018
…nto develop

* 'develop' of github.com:Wechat-Group/weixin-java-tools: (55 commits)
  binarywang#333 修复消息验重问题(mp和cp同时存在的问题)
  修复checkstyle检查出来的部分代码问题
  修改checkstyle配置,以符合IDEA默认的import顺序规则
  重构WxConsts类,对所有常量进行分类整理,便于阅读使用
  规范类命名
  发布临时测试版本2.8.6.BETA
  修复日志
  WxMpTemplateMessage类去掉@NoArgsConstructor注解
  Update .travis.yml
  签名方法里忽略几个自定义key,以免对签名造成干扰
  WxMpTemplateMessage添加默认构造方法
  binarywang#362 修复okhttp实现的qrcode二维码图片下载代码
  binarywang#361 WxMpKefuMessage类的内部类实现序列化接口
  Update readme.md
  优化代码
  发布临时测试版本2.8.5.BETA
  完善修复xmlBean2Map方法
  为兼容之前版本代码,引入两个方法
  修复单元测试
  Update readme.md
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants