Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Version prefix in MQTT topics #24

Closed
hylkevds opened this issue Jun 7, 2017 · 4 comments
Closed

Version prefix in MQTT topics #24

hylkevds opened this issue Jun 7, 2017 · 4 comments
Labels
sensing v1.1 This change should be discussed for v1.1 of the sensing document.

Comments

@hylkevds
Copy link
Contributor

hylkevds commented Jun 7, 2017

This point was raised in the test suit pull request: opengeospatial/ets-sta10#20

In the HTTP requests, all URLs contain the version prefix (v1.0). The MQTT topics do not contain this prefix. This means the MQTT topics are not "version safe". Was this on purpose, or should we add the version prefix to the MQTT topics as well?
@taniakhalafbeigi taniakhalafbeigi added the sensing v1.1 This change should be discussed for v1.1 of the sensing document. label Dec 27, 2018
@hylkevds
Copy link
Contributor Author

What do current implementations do?
FROST wants the version prefix.
How about SensorUP and Gost?

@taniakhalafbeigi
Copy link

We have version prefix like v1.0/Observations.

@bertt
Copy link

bertt commented Apr 24, 2019

In GOST there is no version info in the topic (just like the samples in the spec pdf)

hylkevds added a commit to hylkevds/sensorthings that referenced this issue Aug 1, 2019
@hylkevds
Fixes opengeospatial#24: version prefix in MQTT topics
3c2d340
@hylkevds hylkevds mentioned this issue Aug 1, 2019
Merged
@hylkevds hylkevds mentioned this issue Dec 11, 2019
Merged
@hylkevds
Copy link
Contributor Author

Fixed with the publication of Sensing v1.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
sensing v1.1 This change should be discussed for v1.1 of the sensing document.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bertt @hylkevds @taniakhalafbeigi