wechaty / wechaty-puppet-padplus

DEPRECATED: One puppet based on iPad protocal for Wechaty
https://wechaty.js.org/docs/puppet-services/
315 stars 65 forks source link

Message.Type.Recalled not catching recalled message any more #263

Closed suntong closed 4 years ago

suntong commented 4 years ago

Important:Please file the issue follow the template, or we won't help you to solve the problem.

0. Report Issue Guide

  1. Please run the following command and check whether the problem has been fixed:
npm install wechaty-puppet-padlus@next
  1. Please search in the issue first, and make sure your problem had not been reported before

1. Versions

wechaty@0.40.10
wechaty-puppet-padplus@0.6.3 

2. Describe the bug

My code handles Message.Type.Recalled, and it has been working fine, but lately it is broken.

3. To Reproduce

Steps to reproduce the behavior:

  1. catch & handles Message.Type.Recalled in the code
  2. send a recall message in wx
  3. the Message.Type.Recalled is not caught

4. Expected behavior

the Message.Type.Recalled is caught & handled

5. Actual behavior

the Message.Type.Recalled is not caught & handled

6. Full Output Logs

The new recalled message is of format

<sysmsg type="revokemsg"><revokemsg><session>18323595546@chatroom</session><msgid>1692528799</msgid><newmsgid>5007192124798729796</newmsgid><replacemsg><![CDATA["someone" recalled a message]]></replacemsg></revokemsg></sysmsg>

I believe Message.Type.Recalled is handling the older format.

su-chang commented 4 years ago

Good catch!

It will be fixed in wechaty-puppet-padplus@0.7.27.

Thank you very much.

suntong commented 4 years ago

Fixed in latest padplus@0.7.27