search

drogue-iot / drogue-cloud

Cloud Native IoT
https://drogue.io
Apache License 2.0
114 stars 30 forks source link

fix(endpoints): Add missing UID for downstream events #232

Closed ctron closed 2 years ago

ctron commented 2 years ago

According to our spec, downstream events should have the app/device name, but also the UID. Currently the UID was missing, this change adds it.

Closes: #226

ctron commented 2 years ago

/test

drogue-bot commented 2 years ago

Aye, aye, captain! … Triggered System Test worklow!

drogue-bot commented 2 years ago
Total Passed Failed Ignored Filtered Duration
✅ 147 147 0 0 0 29m 49s

Git: https://github.com/drogue-iot/drogue-cloud @ refs/heads/feature/fix_226_1

Commit: d06951e04013d99b7ab55fa763781520771bf5d1
Author: Jens Reimann <jreimann@redhat.com>
Date: Wed, 16 Feb 2022 11:50:20 -0100

    fix(endpoints): Add missing UID for downstream events

    According to our spec, downstream events should have the app/device
    name, but also the UID. Currently the UID was missing, this change adds
    it.

    Closes: #226

Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/1853047165

Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/02/16/test-run-1853047165.html