search

drogue-iot / drogue-cloud

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

Feature/complex topic names #229

Closed JulianFeinauer closed 2 years ago

JulianFeinauer commented 2 years ago

This pull request adds two different MQTT 3.1 topic name parsing strategies. Currently only the "DrogueV1" is active and there is no way to switch it via configuration just now.

Closes #222.

JulianFeinauer commented 2 years ago

The only thing that is left to be able to finish #222 is to have the possibility to adjust the settings which strategy to use, somewhere. @ctron you have an idea how / where to do that, right?

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 30m 13s

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

Commit: 94e77c5f311d73216905b0504256d66b897dac28
Author: Jens Reimann <jreimann@redhat.com>
Date: Fri, 18 Feb 2022 09:43:29 -0100

    feat(mqtt): Extract dialect information from spec section

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

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

ctron commented 2 years ago

CI looks good too. @JulianFeinauer let me know if/when you want me to merge this.