cockpit icon indicating copy to clipboard operation
cockpit copied to clipboard

Add support for MAVLink Signing

Open rafaellehmkuhl opened this issue 3 months ago • 4 comments

This is needed if you want to make sure nobody can talk to your vehicle without a private key.

More about that here.

rafaellehmkuhl avatar Sep 05 '25 10:09 rafaellehmkuhl

@patrickelectric do you know if mavlink2rest already supports it?

rafaellehmkuhl avatar Sep 05 '25 10:09 rafaellehmkuhl

Related to bluerobotics/BlueOS#3487

ES-Alexander avatar Sep 05 '25 13:09 ES-Alexander

@patrickelectric do you know if mavlink2rest already supports it?

I can help with this answer: it does not, nor does mavlink-server:

  • mavlink2rest depends on rust-mavlink 0.10, which does not support signing
  • mavlink-server depends on our rust-mavlink-codec, which does not support signing

To support it:

  • since we can't simply update mavlink2rest rust-mavlink dependency because it breaks the JSON compatibility, the path is to backport the signing feature to 0.10 on rust-mavlink, and update mavlink2rest to use it
  • for mavlink-server, we'd have to add support for signing on rust-mavlink-codec, and update mavlink-server to use it.

note: other tools, such as mavlink-camera-manager, would also need to be updated.

joaoantoniocardoso avatar Sep 05 '25 13:09 joaoantoniocardoso

@patrickelectric do you know if mavlink2rest already supports it?

I can help with this answer: it does not, nor does mavlink-server: [...]

Yeaaaaah sad I imagined it wasn't going to be that much straightforward but it's clear now that it needs some good work on.

Let me know if at any point you feel like working on it, so we can implement it on cockpit's side as well.

rafaellehmkuhl avatar Sep 05 '25 19:09 rafaellehmkuhl