Don't unsubscribe when subscriber callback throws (#3401)

2016-11-12 22:26:50 +01:00 · 2016-11-12 22:26:50 +01:00 · c57738b9b6
commit c57738b9b6
parent 6b33899239
2 changed files with 0 additions and 2 deletions
--- a/js/src/api/contract/contract.js
+++ b/js/src/api/contract/contract.js
@ -309,7 +309,6 @@ export default class Contract {
          try {
            subscriptions[idx].callback(null, this.parseEventLogs(logs));
          } catch (error) {
-            this.unsubscribe(idx);
            console.error('_sendSubscriptionChanges', error);
          }
        });
--- a/js/src/api/subscriptions/manager.js
+++ b/js/src/api/subscriptions/manager.js
@ -107,7 +107,6 @@ export default class Manager {
      callback(error, data);
    } catch (error) {
      console.error(`Unable to update callback for subscriptionId ${subscriptionId}`, error);
-      this.unsubscribe(subscriptionId);
    }
  }