fix: unexpectedly invoke initSessionAndConnect twice when use it in u…

…seEffect
pjchender · Feb 7, 2020 · 7cd6f86 · 7cd6f86
1 parent 763db72
commit 7cd6f86
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -23,13 +23,13 @@ See our demo site: [react-use-opentok](https://pjchender.github.io/react-use-ope
 Install it with npm:
 
 ```
-npm i react-use-opentok --save
+npm i @opentok/client@2.x react-use-opentok
 ```
 
 Or with yarn:
 
 ```
-yarn add react-use-opentok
+yarn add @opentok/client@2.x react-use-opentok
 ```
 
 > NOTE: remember to install the peer dependency of [@opentok/client](https://www.npmjs.com/package/@opentok/client)

diff --git a/src/index.js b/src/index.js
@@ -68,14 +68,14 @@ const useOpenTok = () => {
   );
 
   const connectSession = useCallback(
-    (token, sessionToConnect = session) =>
+    (token, sessionToConnect) =>
       new Promise((resolve, reject) => {
         if (!token) {
-          reject('[react-use-opentok] token does not exist.');
+          return reject('[react-use-opentok] token does not exist.');
         }
 
         if (!sessionToConnect) {
-          reject('[react-use-opentok] session does not exist.');
+          return reject('[react-use-opentok] session does not exist.');
         }
 
         sessionToConnect.connect(token, error => {
@@ -92,7 +92,7 @@ const useOpenTok = () => {
           }
         });
       }),
-    [action, session]
+    [action]
   );
 
   const initSessionAndConnect = useCallback(