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

mosquitto_topic_matches_sub not matching subscription ending with /+/# #670

Closed
mrdis opened this issue Jan 9, 2018 · 1 comment
Closed

mosquitto_topic_matches_sub not matching subscription ending with /+/# #670

mrdis opened this issue Jan 9, 2018 · 1 comment
Labels
Type: Bug
Milestone
fixes-next

Comments

@mrdis
Copy link

mrdis commented Jan 9, 2018

The function result says that topic "foo/bar" does not match sub "foo/+/#".

I suppose this is wrong, since "foo/bar" matches "foo/bar/#" instead.
@ralight
Copy link
Contributor

ralight commented Jan 9, 2018

I agree that it should match and that it doesn't. Good find!

@ralight ralight added this to the fixes-next milestone Jan 9, 2018
ralight added a commit that referenced this issue Jan 9, 2018
@ralight
[670] Fix topic matching of foo/bar against foo/+/#
b02c1a4 
Thanks to mrdis.

Bug: #670
@ralight ralight closed this as completed Jan 9, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Type: Bug
Projects
None yet
Milestone
fixes-next
Development

No branches or pull requests
2 participants
@ralight @mrdis