ros2 / rmw_zenoh

RMW for ROS 2 using Zenoh as the middleware
Apache License 2.0
185 stars 34 forks source link

Add TLS and compression features by default #180

Closed JEnoch closed 4 months ago

JEnoch commented 4 months ago

Both TLS and compression are important features available in Zenoh. This PR make them build by default for rmw_zenoh.

It also updates a comment wrt. threading and Tokio. This comment became obsolete after the switch of Zenoh from async_std to Tokio.

JEnoch commented 4 months ago

@Yadunund do you think the comment I changed in this PR is good enough for #179 ?

BTW, the compression we enabled in config during our test today was actually not active, because the feature wasn't built... :sweat_smile: