Fix: Log accessory names not futures (#151)

The previous code logged the CompletableFuture, not the name of the accessory.

Fixes #150

Author: MariusVolkhart

CHANGES.md

@@ -1,3 +1,7 @@
+# HAP-Java 2.0.1
+## Fixes
+* Log accessory names instead of futures. [#150](https://github.com/hap-java/HAP-Java/issues/150)
+
 # HAP-Java 2.0.0
 * major refactoring to support optional characteristics
 * structure and names adapted to HAP spec structure and naming.

src/main/java/io/github/hapjava/server/impl/HomekitRoot.java

@@ -83,7 +83,9 @@ public void addAccessory(HomekitAccessory accessory) {
    */
   void addAccessorySkipRangeCheck(HomekitAccessory accessory) {
     this.registry.add(accessory);
-    logger.trace("Added accessory " + accessory.getName());
+    if (logger.isTraceEnabled()) {
+      accessory.getName().thenAccept(name -> logger.trace("Added accessory {}", name));
+    }
     if (started) {
       registry.reset();
       webHandler.resetConnections();
@@ -99,7 +101,9 @@ void addAccessorySkipRangeCheck(HomekitAccessory accessory) {
    */
   public void removeAccessory(HomekitAccessory accessory) {
     this.registry.remove(accessory);
-    logger.trace("Removed accessory " + accessory.getName());
+    if (logger.isTraceEnabled()) {
+      accessory.getName().thenAccept(name -> logger.trace("Removed accessory {}", name));
+    }
     if (started) {
       registry.reset();
       webHandler.resetConnections();