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.

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

MQTT topic routing doesn't correctly reject pub/sub requests #572

Closed
vordimous opened this issue Nov 13, 2023 · 0 comments · Fixed by #573
Closed

MQTT topic routing doesn't correctly reject pub/sub requests #572

vordimous opened this issue Nov 13, 2023 · 0 comments · Fixed by #573
Assignees
Labels
bug Something isn't working

Comments

@vordimous
Copy link
Contributor

Describe the bug
Single topic MQTT setup with no default exit:

    routes:
      - when:
          - publish or subscribe:
              - topic: place/#
  • When a singular route is defined for just publish or subscribe with no default exit, Zilla is still allowing both pub/sub with correct message delivery.
  • Zilla is correctly rejecting publish requests when the topic pattern doesn't match but is still allowing all subscribe requests to succeed without an error.

To Reproduce
Create a single topic MQTT route with no default exit and publish a message to the correct topic while being subscribed. The pub/sub behavior does not adhere to the zilla.yaml.

Expected behavior
when only one pub or sub is defined the appropriate error code should be given for the other.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants