Handle absent course and speed attributes

[robertkleffner]

Some of our CoT payloads don't have course and speed attributes (actually they don't have a track element) in the detail element. This was causing an exception in taky and it wasn't propagating the packets to ATAK clients.

With this change, the packets propagate as we expect (and are used to from FTS).

However, if this is considered invalid CoT and acceptable to not handle, I am willing to withdraw the PR.

[tkuester]

Thanks for the request! This is something I didn't consider when writing taky, and I'm enthralled someone is submitting pull requests. I will be on travel this weekend, but I will take my laptop and see if I can get this merged.

…

On Thu, Jul 20, 2023 at 12:59 PM Robert Kleffner ***@***.***> wrote: Some of our CoT payloads don't have course and speed attributes (actually they don't have a track element) in the detail element. This was causing an exception in taky and it wasn't propagating the packets to ATAK clients. With this change, the packets propagate as we expect (and are used to from FTS). However, if this is considered invalid CoT and acceptable to not handle, I am willing to withdraw the PR. ------------------------------ You can view, comment on, or merge this pull request online at: #88 Commit Summary - a90c1ed <a90c1ed> Handle absent course and speed attributes File Changes (1 file <https://github.com/tkuester/taky/pull/88/files>) - *M* taky/cot/models/takuser.py <https://github.com/tkuester/taky/pull/88/files#diff-b5ab45b82558e89532f181924cb8fd59685e63b384ea95f340c2a8cc56c59562> (8) Patch Links: - https://github.com/tkuester/taky/pull/88.patch - https://github.com/tkuester/taky/pull/88.diff — Reply to this email directly, view it on GitHub <#88>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABQVTMY3GUS7OM6OTMPOBTLXRFPVRANCNFSM6AAAAAA2RW4TSE> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[pusewicz]

Bump @tkuester

[tkuester]

Merged into 65b627b on the next branch!

I did a rebase and squash of the typo commit to try and keep the history a little tidier. Thanks for your submission!