Skip to content

Commit 968cc56

Browse files
authored
Fix nullable issue (openhab#18109)
Signed-off-by: Kai Kreuzer <[email protected]>
1 parent 7870b15 commit 968cc56

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

bundles/org.openhab.binding.nuki/src/main/java/org/openhab/binding/nuki/internal/NukiHandlerFactory.java

+5-2
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
*/
1313
package org.openhab.binding.nuki.internal;
1414

15+
import java.util.UUID;
16+
1517
import org.eclipse.jdt.annotation.NonNullByDefault;
1618
import org.eclipse.jdt.annotation.Nullable;
1719
import org.eclipse.jetty.client.HttpClient;
@@ -106,7 +108,7 @@ public void unregisterHandler(Thing thing) {
106108
}
107109
}
108110

109-
private @Nullable String createCallbackUrl(String id) {
111+
private @Nullable String createCallbackUrl(@Nullable String id) {
110112
final String ipAddress = networkAddressService.getPrimaryIpv4HostAddress();
111113
if (ipAddress == null) {
112114
logger.warn("No network interface could be found to get callback address");
@@ -118,7 +120,8 @@ public void unregisterHandler(Thing thing) {
118120
logger.warn("Cannot find port of the http service.");
119121
return null;
120122
}
121-
String callbackUrl = NukiLinkBuilder.callbackUri(ipAddress, port, id).toString();
123+
String callbackUrl = NukiLinkBuilder
124+
.callbackUri(ipAddress, port, id != null ? id : UUID.randomUUID().toString()).toString();
122125
logger.trace("callbackUrl[{}]", callbackUrl);
123126
return callbackUrl;
124127
}

0 commit comments

Comments
 (0)