Merge pull request #85 from yfre/fix_event_subscription

ccutrer · web-flow · commit efdb3b18932c · 2019-10-17T13:16:50.000-06:00
#84 fix subscription process
diff --git a/pom.xml b/pom.xml
@@ -129,7 +129,7 @@
 			<plugin>
 				<groupId>com.coveo</groupId>
 				<artifactId>fmt-maven-plugin</artifactId>
-				<version>2.6.0</version>
+				<version>2.9</version>
 				<executions>
 					<execution>
 						<goals>
diff --git a/src/main/java/io/github/hapjava/impl/connections/SubscriptionManager.java b/src/main/java/io/github/hapjava/impl/connections/SubscriptionManager.java
@@ -49,11 +49,6 @@ public synchronized void addSubscription(
       LOGGER.info(
           "Added subscription to " + characteristic.getClass() + " for " + connection.hashCode());
     }
-    try {
-      connection.outOfBand(new EventController().getMessage(aid, iid, characteristic));
-    } catch (Exception e) {
-      LOGGER.error("Could not send initial state in response to subscribe event", e);
-    }
   }
 
   public synchronized void removeSubscription(

Original file line number	Diff line number	Diff line change
`@@ -49,11 +49,6 @@ public synchronized void addSubscription(`
`49`	`49`	`LOGGER.info(`
`50`	`50`	`"Added subscription to " + characteristic.getClass() + " for " + connection.hashCode());`
`51`	`51`	`}`
`52`		`- try {`
`53`		`- connection.outOfBand(new EventController().getMessage(aid, iid, characteristic));`
`54`		`- } catch (Exception e) {`
`55`		`- LOGGER.error("Could not send initial state in response to subscribe event", e);`
`56`		`- }`
`57`	`52`	`}`
`58`	`53`
`59`	`54`	`public synchronized void removeSubscription(`