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

The program will be blocked when we use the properties of isAlive Act… #644

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Jan 8, 2021

  1. The program will be blocked when we use the properties of isAlive Act…

    …iveID and InactiveID and all methods relatived to ping undering the case of connectivity of any remote client was terminated suddenly.
    
    That's a big problem when we use the software in real time data transferring and the network was interrupted unexpectedly, so I
    change them to asynchronous method with setting macro definition of __PING_ASYNC at property of project
    
    1.Steps to reproduce:
    Start WebSocketServer
    Start two or more clients to connect to WebSocket Server
    Unplugged the net cable of one of clients' manually
    The property of isAlive ActiveID or InActiveID was used before sending message in WebSocketBehavior or the event of _sweepTimer was triggered at occasion of sending message.
    
    2.Actual result:
    All other rest clients could not receiving any data from WebSocket Server in 10 to 30 seconds until the WebSocket Server receiving the event of that client was closed or offline.
    
    3.Expected result:
    Send messages quickly and continusly
    
    4.Proposed solution:
    Replacing the usage of the ping method with pingAsync
    sinolover committed Jan 8, 2021
    Configuration menu
    Copy the full SHA
    09c07ba View commit details
    Browse the repository at this point in the history