Change default iceCheckingTimeout to 2s

Author: tylerlong

demo/index.ts

@@ -177,10 +177,8 @@ $(() => {
       enableQos: true,
       enableMediaReportLogging: true,
 
-      // in some network environments, the default iceCheckingTimeout of 500ms is not enough
-      // we need more time to gather candidates.
-      // disadvantage of this is that it will take more time to establish the call
-      iceCheckingTimeout: 3000, // 3 seconds
+      // previous default value is 500ms. Now it is 2000ms, so I comment it out
+      // iceCheckingTimeout: 2000,
     };
 
     const urlParams = new URLSearchParams(window.location.search);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ringcentral-web-phone",
-  "version": "1.0.10",
+  "version": "1.0.11",
   "homepage": "https://github.com/ringcentral/ringcentral-web-phone",
   "bugs": {
     "url": "https://github.com/ringcentral/ringcentral-web-phone/issues"

src/index.ts

@@ -129,7 +129,7 @@ export interface WebPhoneOptions {
   enableTurnServers?: boolean;
   /** Max time in milliseconds to be considered when generating ice candidates
    *
-   * default value `2000` when `enableTurnServers` is `true`, otherwise `500`
+   * default value `2000`
    */
   iceCheckingTimeout?: number;
   /** Policy used when generating ice candidates
@@ -366,7 +366,7 @@ export default class WebPhone {
     ];
 
     const sessionDescriptionHandlerFactoryOptions = options.sessionDescriptionHandlerFactoryOptions || {
-      iceGatheringTimeout: options.iceCheckingTimeout || 500,
+      iceGatheringTimeout: options.iceCheckingTimeout || 2000,
       enableDscp: options.enableDscp,
       peerConnectionConfiguration: {
         iceServers,