Skip to content

Wait a few seconds after socket connection for first ping, to prevent…#609

Merged
frankcorneliusmartin merged 1 commit intomainfrom
bugfix/prevent-node-ping-before-proper-startup
Mar 28, 2023
Merged

Wait a few seconds after socket connection for first ping, to prevent…#609
frankcorneliusmartin merged 1 commit intomainfrom
bugfix/prevent-node-ping-before-proper-startup

Conversation

@bartvanb
Copy link
Copy Markdown
Member

… non-connected namespace error

This prevents this error directly after startup:

2023-03-23 14:18:33 - node           - ERROR    - Ping thread had an exception
Traceback (most recent call last):
  File "/vantage6/vantage6-node/vantage6/node/__init__.py", line 812, in __socket_ping_worker
    self.socketIO.emit('ping', namespace='/tasks')
  File "/usr/local/lib/python3.10/site-packages/socketio/client.py", line 393, in emit
    raise exceptions.BadNamespaceError(
socketio.exceptions.BadNamespaceError: /tasks is not a connected namespace.

@frankcorneliusmartin frankcorneliusmartin merged commit c2e380c into main Mar 28, 2023
@frankcorneliusmartin frankcorneliusmartin deleted the bugfix/prevent-node-ping-before-proper-startup branch March 28, 2023 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants