-
-
Notifications
You must be signed in to change notification settings - Fork 429
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
New method to build a valid consumer name for a WEB client #3359
Conversation
Can someone please retrigger the build? Build was aborted after 180 minutes. |
Both fail for the same reason so it is more likely something is wrong with your changes. Maybe resolving itest runbundles fixes it? |
Correct. It is now fixed. |
To associate a HTTP or WebSocket client to a thing. Signed-off-by: Laurent Garnier <lg.hc@free.fr>
6286742
to
b234feb
Compare
Is it possible to have a review of this utility method? |
...les/org.openhab.core.thing/src/main/java/org/openhab/core/thing/util/ThingWebClientUtil.java
Outdated
Show resolved
Hide resolved
...les/org.openhab.core.thing/src/main/java/org/openhab/core/thing/util/ThingWebClientUtil.java
Outdated
Show resolved
Hide resolved
...org.openhab.core.thing/src/test/java/org/openhab/core/thing/util/ThingWebClientUtilTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@wborn : thank you for your review, your comments have been addressed. |
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 now!
) Signed-off-by: Laurent Garnier <lg.hc@free.fr> GitOrigin-RevId: cd45dfe
To associate a HTTP or WebSocket client to a thing.
Signed-off-by: Laurent Garnier lg.hc@free.fr