openimsdk / openim-electron-demo

Instant Messaging web desktop
https://openim.io
Apache License 2.0
159 stars 178 forks source link

The PingMessage sent by electron demo sdk-core cannot be received by openim-server. #80

Open Larry-Arun opened 3 weeks ago

Larry-Arun commented 3 weeks ago

What would you like to share?

electron demo sdk-core 发送的PingMessage,openim-server没有日志,openim-server发送的PingMessage,electron的sdk-core 没有日志,两端发送的PingMessage 互相收不到,都是release-v3.8分支,这是怎么回事

Additional information

都是release-v3.8分支

Bloomingg commented 2 weeks ago

Are you using the wasm version sdk? The heartbeat ping pong in wasm is carried out through the default behavior of the browser and everything is normal.

Larry-Arun commented 6 days ago

yeah,I used the wasm version sdk.I made a mistake before. Openim-server did not send ping to the client because of the line of code on the server side 'c.PlatformID == constant.WebPlatformID'. The server only sends ping when it is a Web. I changed the server code to 'c.PlatformID == constant.WebPlatformID || c.PlatformID == constant.WindowsPlatformID'. When it is an electron Windows client, the server also sends ping. This temporarily fixes the problem, but I think it is more appropriate for the client to send ping. Currently, when the client sends ping, the server does not print logs. The wasm used by the client is compiled from openim-sdk-core.