### Overview of all settings
-| Setting | Description | Default value |
-|:-------------------------|:--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|:--------------|
-| networkInterface | IP address or domain name under which the HomeKit bridge can be reached. If no value is configured, the add-on uses the primary IP address configured for openHAB. If unsure, keep it empty | (none) |
-| port | Port under which the HomeKit bridge can be reached. | 9123 |
-| useOHmDNS | mDNS service is used to advertise openHAB as HomeKit bridge in the network so that HomeKit clients can find it. openHAB has already mDNS service running. This option defines whether the mDNS service of openHAB or a separate service should be used. | false |
-| blockUserDeletion | Blocks HomeKit user deletion in openHAB and as result unpairing of devices. If you experience an issue with accessories becoming non-responsive after some time, try to enable this setting. You can also enable this setting if your HomeKit setup is done and you will not re-pair ios devices. | false |
-| pin | Pin code used for pairing with iOS devices. Apparently, pin codes are provided by Apple and represent specific device types, so they cannot be chosen freely. The pin code 031-45-154 is used in sample applications and known to work. | 031-45-154 |
-| startDelay | HomeKit start delay in seconds in case the number of accessories is lower than last time. This helps to avoid resetting home app in case not all items have been initialised properly before HomeKit integration start. Ignored if `useDummyAccessories` is on. | 30 |
-| useFahrenheitTemperature | Set to true to use Fahrenheit degrees, or false to use Celsius degrees. Note if an item has a QuantityType as its state, this configuration is ignored and it's always converted properly. | false |
-| thermostatTargetModeCool | Word used for activating the cooling mode of the device (if applicable). It can be overwritten at item level. | CoolOn |
-| thermostatTargetModeHeat | Word used for activating the heating mode of the device (if applicable). It can be overwritten at item level. | HeatOn |
-| thermostatTargetModeAuto | Word used for activating the automatic mode of the device (if applicable). It can be overwritten at item level. | Auto |
-| thermostatTargetModeOff | Word used to set the thermostat mode of the device to off (if applicable). It can be overwritten at item level. | Off |
-| name | Name under which this HomeKit bridge is announced on the network. This is also the name displayed on the iOS device when searching for available bridges. | openHAB |
-| instances | Defines how many bridges to expose. Necessary if you have more than 149 accessories. Accessories must be assigned to additional instances via metadata. Additional bridges will use incrementing port numbers. | 1 |
-| useDummyAccessories | When an accessory is missing, substitute a dummy in its place instead of removing it. See [Dummy Accessories](#dummy-accessories). | false |
+| Setting | Description | Default value |
+|:-------------------------|:-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|:---------------------|
+| networkInterface | IP address or domain name under which the HomeKit bridge can be reached. If no value is configured, the add-on uses the primary IP address configured for openHAB. If unsure, keep it empty | (none) |
+| port | Port under which the HomeKit bridge can be reached. | 9123 |
+| useOHmDNS | mDNS service is used to advertise openHAB as HomeKit bridge in the network so that HomeKit clients can find it. openHAB has already mDNS service running. This option defines whether the mDNS service of openHAB or a separate service should be used. | false |
+| blockUserDeletion | Blocks HomeKit user deletion in openHAB and as result unpairing of devices. If you experience an issue with accessories becoming non-responsive after some time, try to enable this setting. You can also enable this setting if your HomeKit setup is done and you will not re-pair ios devices. | false |
+| pin | Pin code used for pairing with iOS devices. Apparently, pin codes are provided by Apple and represent specific device types, so they cannot be chosen freely. The pin code 031-45-154 is used in sample applications and known to work. | 031-45-154 |
+| useFahrenheitTemperature | Set to true to use Fahrenheit degrees, or false to use Celsius degrees. Note if an item has a QuantityType as its state, this configuration is ignored and it's always converted properly. | false |
+| thermostatTargetModeCool | Word used for activating the cooling mode of the device (if applicable). It can be overwritten at item level. | CoolOn |
+| thermostatTargetModeHeat | Word used for activating the heating mode of the device (if applicable). It can be overwritten at item level. | HeatOn |
+| thermostatTargetModeAuto | Word used for activating the automatic mode of the device (if applicable). It can be overwritten at item level. | Auto |
+| thermostatTargetModeOff | Word used to set the thermostat mode of the device to off (if applicable). It can be overwritten at item level. | Off |
+| name | Name under which this HomeKit bridge is announced on the network. This is also the name displayed on the iOS device when searching for available bridges. | openHAB |
+| instances | Defines how many bridges to expose. Necessary if you have more than 149 accessories. Accessories must be assigned to additional instances via metadata. Additional bridges will use incrementing port numbers. | 1 |
+| useDummyAccessories | When an accessory is missing, substitute a dummy in its place instead of removing it. See [Dummy Accessories](#dummy-accessories). | false |
## Item Configuration
import java.util.List;
import java.util.Map;
import java.util.concurrent.ScheduledExecutorService;
-import java.util.concurrent.TimeUnit;
import javax.jmdns.JmDNS;
import org.openhab.core.net.CidrAddress;
import org.openhab.core.net.NetworkAddressChangeListener;
import org.openhab.core.net.NetworkAddressService;
+import org.openhab.core.service.ReadyMarker;
+import org.openhab.core.service.ReadyMarkerFilter;
+import org.openhab.core.service.ReadyService;
+import org.openhab.core.service.StartLevelService;
import org.openhab.core.storage.Storage;
import org.openhab.core.storage.StorageService;
import org.openhab.io.homekit.Homekit;
Constants.SERVICE_PID + "=org.openhab.homekit", "port:Integer=9123" })
@ConfigurableService(category = "io", label = "HomeKit Integration", description_uri = "io:homekit")
@NonNullByDefault
-public class HomekitImpl implements Homekit, NetworkAddressChangeListener {
+public class HomekitImpl implements Homekit, NetworkAddressChangeListener, ReadyService.ReadyTracker {
private final Logger logger = LoggerFactory.getLogger(HomekitImpl.class);
private final StorageService storageService;
private final ConfigurationAdmin configAdmin;
private final ItemRegistry itemRegistry;
private final MetadataRegistry metadataRegistry;
+ private final ReadyService readyService;
private final List<HomekitAuthInfoImpl> authInfos = new ArrayList<>();
private HomekitSettings settings;
private final List<HomekitServer> homekitServers = new ArrayList<>();
private final List<HomekitRoot> bridges = new ArrayList<>();
private MDNSClient mdnsClient;
+ private int currentStartLevel = 0;
private final List<HomekitChangeListener> changeListeners = new ArrayList<>();
@Activate
public HomekitImpl(@Reference StorageService storageService, @Reference ItemRegistry itemRegistry,
@Reference NetworkAddressService networkAddressService, @Reference MetadataRegistry metadataRegistry,
- @Reference ConfigurationAdmin configAdmin, @Reference MDNSClient mdnsClient, Map<String, Object> properties)
+ @Reference ConfigurationAdmin configAdmin, @Reference MDNSClient mdnsClient,
+ @Reference ReadyService readyService, Map<String, Object> properties)
throws IOException, InvalidAlgorithmParameterException {
this.storageService = storageService;
this.networkAddressService = networkAddressService;
this.mdnsClient = mdnsClient;
this.itemRegistry = itemRegistry;
this.metadataRegistry = metadataRegistry;
+ this.readyService = readyService;
networkAddressService.addNetworkAddressChangeListener(this);
- try {
- startHomekitServer();
- } catch (IOException | InvalidAlgorithmParameterException e) {
- logger.warn("cannot activate HomeKit binding. {}", e.getMessage());
- throw e;
- }
+ readyService.registerTracker(this, new ReadyMarkerFilter().withType(StartLevelService.STARTLEVEL_MARKER_TYPE));
}
private HomekitSettings processConfig(Map<String, Object> properties) {
|| oldSettings.instances != settings.instances) {
// the HomeKit server settings changed. we do a complete re-init
stopHomekitServer();
- startHomekitServer();
+ if (currentStartLevel >= StartLevelService.STARTLEVEL_STATES) {
+ startHomekitServer();
+ }
} else {
for (HomekitChangeListener changeListener : changeListeners) {
changeListener.updateSettings(settings);
}
}
+ @Override
+ public synchronized void onReadyMarkerAdded(ReadyMarker readyMarker) {
+ int newLevel = Integer.parseInt(readyMarker.getIdentifier());
+ currentStartLevel = newLevel;
+
+ if (newLevel >= StartLevelService.STARTLEVEL_STATES) {
+ try {
+ startHomekitServer();
+ } catch (IOException | InvalidAlgorithmParameterException e) {
+ logger.warn("could not initialize HomeKit bridge: {}", e.getMessage());
+ }
+ }
+ }
+
+ @Override
+ @SuppressWarnings("PMD.EmptyWhileStmt")
+ public synchronized void onReadyMarkerRemoved(ReadyMarker readyMarker) {
+ int newLevel = Integer.parseInt(readyMarker.getIdentifier());
+
+ if (currentStartLevel > newLevel) {
+ while (newLevel-- > 0 && !readyService
+ .isReady(new ReadyMarker(StartLevelService.STARTLEVEL_MARKER_TYPE, Integer.toString(newLevel)))) {
+ }
+ currentStartLevel = newLevel;
+ if (currentStartLevel < StartLevelService.STARTLEVEL_STATES) {
+ stopHomekitServer();
+ }
+ }
+ }
+
private HomekitRoot startBridge(HomekitServer homekitServer, HomekitAuthInfoImpl authInfo,
HomekitChangeListener changeListener, int instance) throws IOException {
String name = settings.name;
final int lastAccessoryCount = changeListener.getLastAccessoryCount();
int currentAccessoryCount = changeListener.getAccessories().size();
- if (!settings.useDummyAccessories && currentAccessoryCount < lastAccessoryCount) {
- logger.debug(
- "it looks like not all items were initialized yet. Old configuration had {} accessories, the current one has only {} accessories. Delay HomeKit bridge start for {} seconds.",
- lastAccessoryCount, currentAccessoryCount, settings.startDelay);
- scheduler.schedule(() -> {
- if (currentAccessoryCount < lastAccessoryCount) {
- // the number of items is still different, maybe it is desired.
- // make new configuration revision.
- changeListener.makeNewConfigurationRevision();
- }
- bridge.start();
- }, settings.startDelay, TimeUnit.SECONDS);
- } else { // start bridge immediately.
- bridge.start();
- }
+ bridge.start();
return bridge;
}
@Override
public synchronized void onChanged(final List<CidrAddress> added, final List<CidrAddress> removed) {
logger.trace("HomeKit bridge reacting on network interface changes.");
+ if (currentStartLevel < StartLevelService.STARTLEVEL_STATES) {
+ return;
+ }
removed.forEach(i -> {
logger.trace("removed interface {}", i.getAddress().toString());
if (i.getAddress().equals(networkInterface)) {
projects / openhab-addons.git / commitdiff