`MsgUtcTime` implementation of `gps_time` is not checking status flags about libsbp

[pcrumley]

Hi @marcomaida94 thank you for bringing this issue to our attention. We haven't had a chance to look at it, but with a quick look it seems very similar to the other gps_time status flag issue and hopefully has a similar fix.

We'll let you know when a fix has been implemented.

[pcrumley]

@marcomaida94 we had a follow-up PR right after that which closed some additional gaps including this one: #1376

i think this should be fixed. let's close it and then you can re-open if you run into the issue again.

[marcomaida94]

Good morning, did you have time to take a look at this?

Thanks!

[marcomaida94]

Hello, Patrick. I agree with you that it is hopefully quite similar.

Thanks, will keep an eye on it!
Marco

[pcrumley]

@marcomaida94 I haven't tried to reproduce this bug yet, but are you using the latest version of libsbp 5.0.4? It looks like this line here:

libsbp/rust/sbp/src/messages/navigation.rs

Line 7229 in 5457e5d

if self.time_source().ok()? == TimeSource::None {

should solve the issue.

[marcomaida94]

Hey @pcrumley , we might have different versions installed on different machines, so it might be that it was recorded not on a fully up-to-date version, though I was expecting this to come from a machine that already integrated #1363.

If you think this should be solved on your side, I am ok to close this issue and re-open in case we experience it again.

Thanks for looking into this!