feat: simplified peerjs v1.4.7 version support

refs: #CAR-697

Author: boris-burgos

build.sh

@@ -3,9 +3,9 @@
 exec 2>&1
 
 rm -rf dist/ peerjs/
-git clone https://github.com/Codeko/peerjs.git --branch v1.4.7-react-native --single-branch peerjs
+git clone https://github.com/peers/peerjs --branch v1.4.7 --single-branch peerjs
 cd peerjs
-# git apply ../decoupling.diff
+git apply ../decoupling.diff
 ../node_modules/.bin/parcel build --no-source-maps lib/exports.ts -d ../dist --out-file peerjs.min.js
 cd ../
 cat imports.js dist/peerjs.min.js > dist/react-native-peerjs.js

decoupling.diff

@@ -1,139 +1,20 @@
-diff --git a/lib/dataconnection.ts b/lib/dataconnection.ts
-index aec3c05..f0ca925 100644
---- a/lib/dataconnection.ts
-+++ b/lib/dataconnection.ts
-@@ -58,7 +58,7 @@ export class DataConnection extends BaseConnection implements IDataConnection {
-       this.options.connectionId || DataConnection.ID_PREFIX + util.randomToken();
-
-     this.label = this.options.label || this.connectionId;
--    this.serialization = this.options.serialization || SerializationType.Binary;
-+    this.serialization = this.options.serialization || SerializationType.JSON;
-     this.reliable = !!this.options.reliable;
-
-     this._encodingQueue.on('done', (ab: ArrayBuffer) => {
-diff --git a/lib/exports.ts b/lib/exports.ts
-index 5772d02..63a57e3 100644
---- a/lib/exports.ts
-+++ b/lib/exports.ts
-@@ -7,7 +7,3 @@ export const peerjs = {
- };
-
- export default Peer;
--
--(<any>window).peerjs = peerjs;
--/** @deprecated Should use peerjs namespace */
--(<any>window).Peer = Peer;
-diff --git a/lib/negotiator.ts b/lib/negotiator.ts
-index e2b3df9..c48efa3 100644
---- a/lib/negotiator.ts
-+++ b/lib/negotiator.ts
-@@ -142,10 +142,11 @@ export class Negotiator {
-     // MEDIACONNECTION.
-     logger.log("Listening for remote stream");
-
--    peerConnection.ontrack = (evt) => {
--      logger.log("Received remote stream");
-+    // react-native-webrtc implements the old API.
-+    peerConnection.onaddstream = (evt) => {
-+      logger.log("Received remote stream", evt);
-
--      const stream = evt.streams[0];
-+      const stream = evt.stream;
-       const connection = provider.getConnection(peerId, connectionId);
-
-       if (connection.type === ConnectionType.Media) {
-@@ -168,7 +169,7 @@ export class Negotiator {
-     this.connection.peerConnection = null;
-
-     //unsubscribe from all PeerConnection's events
--    peerConnection.onicecandidate = peerConnection.oniceconnectionstatechange = peerConnection.ondatachannel = peerConnection.ontrack = () => { };
-+    peerConnection.onicecandidate = peerConnection.oniceconnectionstatechange = peerConnection.ondatachannel = peerConnection.onaddstream = () => { };
-
-     const peerConnectionNotClosed = peerConnection.signalingState !== "closed";
-     let dataChannelNotClosed = false;
-@@ -340,15 +341,14 @@ export class Negotiator {
-   ): void {
-     logger.log(`add tracks from stream ${stream.id} to peer connection`);
-
--    if (!peerConnection.addTrack) {
-+    // react-native-webrtc implements the old API.
-+    if (!peerConnection.addStream) {
-       return logger.error(
--        `Your browser does't support RTCPeerConnection#addTrack. Ignored.`
-+        `Your browser does't support RTCPeerConnection#addStream. Ignored.`
-       );
-     }
-
--    stream.getTracks().forEach(track => {
--      peerConnection.addTrack(track, stream);
--    });
-+    peerConnection.addStream(stream);
-   }
-
-   private _addStreamToMediaConnection(
-diff --git a/lib/peer.ts b/lib/peer.ts
-index 8f11659..1af0753 100644
---- a/lib/peer.ts
-+++ b/lib/peer.ts
-@@ -111,11 +111,6 @@ export class Peer extends EventEmitter {
-     };
-     this._options = options;
-
--    // Detect relative URL host.
--    if (this._options.host === "/") {
--      this._options.host = window.location.hostname;
--    }
--
-     // Set path correctly.
-     if (this._options.path) {
-       if (this._options.path[0] !== "/") {
 diff --git a/lib/supports.ts b/lib/supports.ts
-index 1801188..db1dc70 100644
+index 902b61a..3ee6d64 100644
 --- a/lib/supports.ts
 +++ b/lib/supports.ts
-@@ -1,5 +1,3 @@
--import { webRTCAdapter } from './adapter';
--
- export const Supports = new class {
-   readonly isIOS = ['iPad', 'iPhone', 'iPod'].includes(navigator.platform);
-   readonly supportedBrowsers = ['firefox', 'chrome', 'safari'];
-@@ -28,36 +26,15 @@ export const Supports = new class {
-   }
+@@ -17,6 +17,7 @@ export const Supports = new (class {
+ 	}
 
-   getBrowser(): string {
--    return webRTCAdapter.browserDetails.browser;
-+    return 'chrome';
-   }
+ 	isBrowserSupported(): boolean {
++		if(navigator.product === 'ReactNative') return true;
+ 		const browser = this.getBrowser();
+ 		const version = this.getVersion();
 
-   getVersion(): number {
--    return webRTCAdapter.browserDetails.version || 0;
-+    return this.minChromeVersion;
-   }
+@@ -41,6 +42,7 @@ export const Supports = new (class {
+ 	}
 
-   isUnifiedPlanSupported(): boolean {
--    const browser = this.getBrowser();
--    const version = webRTCAdapter.browserDetails.version || 0;
--
--    if (browser === 'chrome' && version < 72) return false;
--    if (browser === 'firefox' && version >= 59) return true;
--    if (!window.RTCRtpTransceiver || !('currentDirection' in RTCRtpTransceiver.prototype)) return false;
--
--    let tempPc: RTCPeerConnection;
--    let supported = false;
--
--    try {
--      tempPc = new RTCPeerConnection();
--      tempPc.addTransceiver('audio');
--      supported = true;
--    } catch (e) { }
--    finally {
--      if (tempPc) {
--        tempPc.close();
--      }
--    }
--
--    return supported;
-+    return false;
-   }
+ 	isUnifiedPlanSupported(): boolean {
++		if(navigator.product === 'ReactNative') return true;
+ 		const browser = this.getBrowser();
+ 		const version = webRTCAdapter.browserDetails.version || 0;
 
-   toString(): string {