-
Notifications
You must be signed in to change notification settings - Fork 669
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
NOISSUE - Fix subtopic regex and restrict empty subtopic parts #659
Conversation
Codecov Report
@@ Coverage Diff @@
## master #659 +/- ##
==========================================
- Coverage 86.66% 86.29% -0.37%
==========================================
Files 62 62
Lines 3779 3722 -57
==========================================
- Hits 3275 3212 -63
- Misses 346 353 +7
+ Partials 158 157 -1
Continue to review full report at Codecov.
|
8fa091a
to
c0ec95b
Compare
Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
674ab68
to
bf5bc3b
Compare
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…ch#659) * Fix subtopic regex and restrict empty subtopic parts Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Update mqtt adapter subtopic processing Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Return old MQTT adapter implementation Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Allow dots and handle empty parts Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Add malformed subtopic error encoding Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Fix MQTT topic validation Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Fix MQTT topic parsing Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Fix subtopic filtering Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
* Fix subtopic regex and restrict empty subtopic parts Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Update mqtt adapter subtopic processing Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Return old MQTT adapter implementation Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Allow dots and handle empty parts Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Add malformed subtopic error encoding Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Fix MQTT topic validation Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Fix MQTT topic parsing Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com> * Fix subtopic filtering Signed-off-by: Aleksandar Novakovic <aleksandar.novakovic@mainflux.com>
What does this do?
It restricts publish and subscribe to subtopic that contains
*
or>
inside topic part.Have you included tests for your changes?
No.
Did you document any new/modified functionality?
No.