search

drogue-iot / drogue-ttn

Rust mappings for APIs of *The Things Network*
Apache License 2.0
3 stars 1 forks source link

Make snr field in Metadata struct optional #5

Closed dbrgn closed 2 years ago

dbrgn commented 2 years ago

I found another optional field in the wild: The snr field is sometimes missing.

Since this seems to be a bit of whack-a-mole: Is there any specification on how these metadata JSONs look like, and which fields may be missing? If not, can we find the source code that generates it?

ctron commented 2 years ago

It indeed is. When I started it, the only way to be "sure" was to inspect the actual content. The documentation I found was not always correct, and the code was changing frequently.

I don't have issue with frequent PRs :)

dbrgn commented 2 years ago

Thanks!