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.
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
Topic branch renaming in bridge not consistent #40
Comments
We could add a case when topic is the remote/local prefix without the ending slash and do a substitution without the ending slash. This would fix this issue. But I would like to suggest something slightly different: instead of just prepending the prefix as a string to the topic, I propose to consider the prefix as a hierarchy and prepend this hierarchy to the topic. All of the following would yield the same result:
Under the hood, the ending slash if present will be str This would break rules that don't have slash between prefix and topic:
The new proposed solution would subscribe to "remote/pre/fix/#" while currently is would subscribe to "remote/prefix/#". It would also break rules that don't have same slash on both side (remote vs local):
The new proposed solution would remap topic $SYS/broker/clients/total to test/mosquitto/org/clients/total while currently it is remapped to test/mosquitto/orgclients/total. For this case, I'm think the proposed behavior is better (the example is taken from manpage and the manpage said that result is the proposed behavior). If this proposed solution is accepted, I could work on implementing it for the develop branch. In the meantime, I've added tests (on fixes branch) to make sure no regression happen. Check that exhibit this issue is commented to avoid failing test. |
migrated from Bugzilla #473286
status NEW severity normal in component Mosquitto for 1.4
Reported in version 1.4 on platform PC
Assigned to: Roger Light
On 2015-07-22 10:07:25 -0400, Jan-Piet Mens wrote:
On 2015-07-22 10:13:04 -0400, Jan-Piet Mens wrote:
On 2015-07-25 18:02:50 -0400, Roger Light wrote:
The text was updated successfully, but these errors were encountered: