From a05b23347f979a87525f143e030f679c3b227b23 Mon Sep 17 00:00:00 2001 From: pabloFuente Date: Wed, 21 Nov 2018 15:10:44 +0100 Subject: [PATCH] openvidu-browser: not supported browser detection improved --- openvidu-browser/src/OpenVidu/OpenVidu.ts | 5 ++++- openvidu-browser/src/OpenVidu/Session.ts | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/openvidu-browser/src/OpenVidu/OpenVidu.ts b/openvidu-browser/src/OpenVidu/OpenVidu.ts index 6d3f78f9..a832d5ab 100644 --- a/openvidu-browser/src/OpenVidu/OpenVidu.ts +++ b/openvidu-browser/src/OpenVidu/OpenVidu.ts @@ -277,12 +277,15 @@ export class OpenVidu { checkSystemRequirements(): number { const browser = platform.name; const family = platform.os!!.family; + const userAgent = !!platform.ua ? platform.ua : navigator.userAgent; // Reject iPhones and iPads if not Safari ('Safari' also covers Ionic for iOS) - if (family === 'iOS' && browser !== 'Safari') { + if (family === 'iOS' && (browser !== 'Safari' || userAgent.indexOf('CriOS') !== -1 || userAgent.indexOf('FxiOS') !== -1)) { return 0; } + // Accept: Chrome (desktop and Android), Firefox (desktop and Android), Opera (desktop and Android), + // Safari (OSX and iOS), Ionic (Android and iOS) if ( (browser !== 'Safari') && (browser !== 'Chrome') && (browser !== 'Chrome Mobile') && diff --git a/openvidu-browser/src/OpenVidu/Session.ts b/openvidu-browser/src/OpenVidu/Session.ts index cb782052..54ebea37 100644 --- a/openvidu-browser/src/OpenVidu/Session.ts +++ b/openvidu-browser/src/OpenVidu/Session.ts @@ -152,7 +152,7 @@ export class Session implements EventDispatcher { reject(error); }); } else { - reject(new OpenViduError(OpenViduErrorName.BROWSER_NOT_SUPPORTED, 'Browser ' + platform.name + ' ' + platform.version + ' is not supported in OpenVidu')); + reject(new OpenViduError(OpenViduErrorName.BROWSER_NOT_SUPPORTED, 'Browser ' + platform.name + ' for ' + platform.os!!.family + ' is not supported in OpenVidu')); } }); }