Additional logging & pubnub origin

ringcentral · Apr 28, 2020 · 7ab75bd · 7ab75bd
1 parent 5659f76
commit 7ab75bd
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/api-test/package.json b/api-test/package.json
@@ -3,6 +3,6 @@
 	"name": "@ringcentral/sdk-api-test",
 	"private": true,
 	"dependencies": {
-		"soap": "0.23.0"
+		"soap": "0.31.0"
 	}
 }
diff --git a/demo/src/core.html b/demo/src/core.html
@@ -103,6 +103,9 @@ <h1>
                     renewHandicapMs: 14 * 60 * 1000 // set quicker subscription refresh
                 });
 
+                subscription.on(subscription.events.status, function(msg) {
+                    console.log(msg);
+                });
                 subscription.on(subscription.events.notification, function(msg) {
                     cb(msg.body);
                 });

diff --git a/subscriptions/package.json b/subscriptions/package.json
@@ -18,7 +18,7 @@
     "start:webpack": "npm run build:webpack -- --watch"
   },
   "dependencies": {
-    "pubnub": "4.21.6"
+    "pubnub": "^4.27.6"
   },
   "devDependencies": {
     "@ringcentral/sdk": "*",

diff --git a/subscriptions/src/subscription/Subscription.ts b/subscriptions/src/subscription/Subscription.ts
@@ -30,6 +30,7 @@ export enum events {
     subscribeError = 'subscribeError',
     automaticRenewSuccess = 'automaticRenewSuccess',
     automaticRenewError = 'automaticRenewError',
+    status = 'status',
 }
 
 export default class Subscription extends SDK.EventEmitter {
@@ -337,10 +338,12 @@ export default class Subscription extends SDK.EventEmitter {
                 ssl: true,
                 restore: true,
                 subscribeKey: subscriberKey,
+                origin: 'ringcentral.pubnubapi.com',
+                keepAlive: true,
             });
 
             this._pubnub.addListener({
-                status: statusEvent => {},
+                status: statusEvent => this.emit(this.events.status, statusEvent),
                 message: m => this._notify(m.message),
             });
         }