-
Notifications
You must be signed in to change notification settings - Fork 18
fix: fix quote message campatibility #50
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
Conversation
建议将这堆xml单独放置 |
@0honus0 好的,已挪到单独文件中 |
更新到此commit之后,对方引用的消息,在tg没有了引用的显示 |
@jiz4oh 确认下问题吧 |
对方引用的 tg 发送的消息还是其他人的消息? |
对方引用我方通过tg发送的消息,在tg端不显示之前那种有长分隔符的引用样式,只显示对方的消息。但是我方在wx端查看,是有引用的 |
这个 pr 影响的功能是 tg 端回复其他人消息。理论上不影响其他人在微信端的任何操作 |
@jiz4oh |
我尝试了9小时之前的commit,出现了更加奇怪的错误,tg端回复wx发来的消息,提示 NameError("name 'text_to_send' is not defined") |
退回到87c07fe9e9cf55b3ecbe04381c2a5c05fde12d3c的commit,发现对方手机端引用消息,我方tg可以看到引用的情况,但是对方用电脑端引用,则依然无效。 |
这一坨不是一个正常的微信 xml 消息结构,请确认你是否复制粘贴正确以及对方是否是正常微信客户端 |
该问题已提交修复,感谢报告 |
这两个日志里的 message 中依然不是一个有效的微信 xml 消息体 ![]() 日志中的消息会在 efb-wechat-comwechat-slave/efb_wechat_comwechat_slave/MsgDeco.py Lines 240 to 241 in 93295f0
抛出错误,无法进入到 traceback 中的 |
似乎是 github 的问题,会移除部分 xml 内容 @phoenixxie0 你是否在 tg 群里,从群里将对应的日志文件发给我呢 |
![]() @phoenixxie0 对方的客户端少了 chatusr 这个数据,请问是否知道对方客户端版本号,我稍后提交一个兼容性修复 |
对方是weix的Windows桌面版3.9.12.55.我尝试了几个版本,好像3.9.1X.xx这一个版本下都有同样的问题? |
有可能,目前已知微信桌面端的实现和移动端非常不一样,少了很多移动端的兜底逻辑。
感谢你的反馈🙏,经研究该问题由以前的提交引入,不为当前这个 pr 所引入,如果后续有任何新的问题请提交一个新的 issue |
微信在电脑端存在 bug,不同版本对引用消息的处理逻辑存在区别。同一条消息的显示效果可能会存在如下区别:
mac Version. 3.6.1 (24207):
mac Version 3.8.10 (28633):
mac Version 4.0.6.19:
现在将引用消息的 发送人名称,和 xml 文本传入以保证兼容性
xml 文本的获取依赖于 etm 数据库存储 vendor_specific 字段,依赖于这个 pr