-
Notifications
You must be signed in to change notification settings - Fork 670
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 Websocket Adapter #2700
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2700 +/- ##
==========================================
- Coverage 42.26% 42.01% -0.26%
==========================================
Files 350 14 -336
Lines 47903 726 -47177
==========================================
- Hits 20247 305 -19942
+ Misses 25450 414 -25036
+ Partials 2206 7 -2199 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
55c53cb
to
ddc40fd
Compare
192b2e0
to
293038d
Compare
f3dcaef
to
e9d81ca
Compare
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.
@rodneyosodo Please update this one.
b4d2337
to
791a90e
Compare
Signed-off-by: Rodney Osodo <[email protected]>
2ff7ed9
to
6e03fed
Compare
Signed-off-by: Rodney Osodo <[email protected]>
6e03fed
to
e6889c6
Compare
What type of PR is this?
This is a bug fix PR
What does this do?
This pull request primarily addresses websocket authentication issues by refining the authentication flow in the websocket handlers and includes broader improvements for the MQTT adapter. It introduces new configuration parameters for the MQTT service, updates docker configurations and dependencies.
Which issue(s) does this PR fix/relate to?
No issue
Have you included tests for your changes?
Tested manually
Did you document any new/modified feature?
Notes
To be merged after #2695