Fix node clean up

[nahueespinosa]

This patch fixes the clean up order of node resources. Without it, the node would hang when shutting down due to circular dependencies with shared pointers.

Fixes #67.

[ivanpauno]

LGTM!

[nahueespinosa]

For future reference, in case someone wants to take a closer look into this. Here are some observations:

• tf2_ros::MessageFilter<sensor_msgs::msg::LaserScan> holds a shared_ptr constructed from get_node_logging_interface().
• Depending on whether we reset the laser_scan_filter_ on deactivate, we get different usage counts for the logging interface shared pointer at the time the node destructor finished executing:
  • 6 if laser_scan_filter_ was cleaned up properly
  • 7 otherwise
• Having this extra instance after returning from the node destructor causes the node base class destructor to hang.