thin-edge / thin-edge.io

The open edge framework for lightweight IoT devices
https://thin-edge.io
Apache License 2.0
211 stars 55 forks source link

Do not panic changing ownership of device cert to unknown user #2875

Closed didier-wenzek closed 1 month ago

didier-wenzek commented 1 month ago

Proposed changes

Avoid to panic when running tedge connect c8y on a device where there is no mosquitto user.

https://github.com/thin-edge/thin-edge.io/issues/2867

Types of changes

Paste Link to the issue

https://github.com/thin-edge/thin-edge.io/issues/2867

Checklist

Further comments

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 77.8%. Comparing base (2fad9e8) to head (1893d1f). Report is 15 commits behind head on main.

Additional details and impacted files | [Files](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2875?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge) | Coverage Δ | | |---|---|---| | [crates/core/tedge/src/cli/connect/command.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2875?src=pr&el=tree&filepath=crates%2Fcore%2Ftedge%2Fsrc%2Fcli%2Fconnect%2Fcommand.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2NvcmUvdGVkZ2Uvc3JjL2NsaS9jb25uZWN0L2NvbW1hbmQucnM=) | `0.0% <0.0%> (ø)` | | ... and [22 files with indirect coverage changes](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2875/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge)
github-actions[bot] commented 1 month ago

Robot Results

:white_check_mark: Passed :x: Failed :next_track_button: Skipped Total Pass % :stopwatch: Duration
433 0 3 433 100 1h7m44.072434999s