search

OpenTelecom / WKWebViewRTC

MIT License
94 stars 30 forks source link

"Audio and video error: Not implemented" with Jitsi Meet #12

Closed vnnvanhuong closed 3 years ago

vnnvanhuong commented 3 years ago

Hi there,

Firstly, thank you for your effort in working on this library. I intended to use Jitsi Meet as a video call feature on my iOS app. Without using this library, I got the error Audio and video error: undefined is not an object 'navigator.mediaDevices.getUserMedia'. I noticed that it is because WebRTC is currently not supported in WKWebView.

The error was gone when applying to this library. Yay!

Unfortunately, I got another error as below

IMG_3288A909BEC1-1

Here is my demo implementation https://github.com/vnnvanhuong/jitsi-meet-on-wkwebview-demo

Thank you Huong

noahmehl commented 3 years ago

@vnnvanhuong if you could provide:

Thanks!

vnnvanhuong commented 3 years ago

Hi @noahmehl

I created a room on this website https://meet.jit.si. I pushed my demo project into GitHub as mentioned above.

When I tried to inspect the error I noticed that the error came from two methods getConstrainst and applyConstrainst since they are not implemented yet (https://github.com/OpenTelecom/WKWebViewRTC/blob/master/WKWebViewRTC/Js/src/MediaStreamTrack.js).

Thank you

noahmehl commented 3 years ago

@vnnvanhuong My guess is that getConstraint() and applyConstraints() are either not implemented or not working. I have not tested this.

hthetiot commented 3 years ago

@vnnvanhuong My guess is that getConstraint() and applyConstraints() are either not implemented or not working. I have not tested this.

Fixed in 6.0.17 https://github.com/cordova-rtc/cordova-plugin-iosrtc/commit/7058105304a05563414149423c692881177814b0

hthetiot commented 3 years ago

navigator.mediaDevices.getUserMedia is only available from iOS 14.3. Check: https://webkit.org/blog/11353/mediarecorder-api/

This is useless information that is not relevant to the issue for couple of reasons:

  1. It only it available on https since iOS 14.3
  2. This library does provide gUM all the way to ios 10
noahmehl commented 3 years ago

@vnnvanhuong these methods should now be implemented in master. Please retest.

noahmehl commented 3 years ago

@vnnvanhuong did your issues get resolved?