Open cottsay opened 4 years ago
dynamic_reconfigure.client.Client
rosrun dynamic_reconfigure dynparam set ...
I'm not sure it would be safe to add these checks post-release, but I thought the issue should be raised.
Inspired by https://github.com/ros-visualization/rqt_reconfigure/pull/80#issuecomment-642733360
dynamic_reconfigure.client.Client
class currently propagates changes to enum values that are not one of the defined values for the enum.rosrun dynamic_reconfigure dynparam set ...
allows a user to set an invalid value.I'm not sure it would be safe to add these checks post-release, but I thought the issue should be raised.
Inspired by https://github.com/ros-visualization/rqt_reconfigure/pull/80#issuecomment-642733360