namgk / node-red-contrib-ros

Node-RED nodes for connecting to ROS bridge
MIT License
15 stars 15 forks source link

Remove duplicated retry handling #2

Open kazuhitoyokoi opened 4 years ago

kazuhitoyokoi commented 4 years ago

In the current implementation, Node-RED seems to crash when the node retries connections because of huge memory usage. The problem occurs from duplicated retry handling (When the node is disconnected, both error and close events occurred and both handlings try to make a connection again). Because single handling is enough for retrying, I removed the handling in the error event.