search

songz / cordova-plugin-opentok

Cordova Plugin for OpenTok - add webrtc video to your iOS or Android App
Other
162 stars 270 forks source link

Only one connected session at once #167

Open Pentiado opened 9 years ago

Pentiado commented 9 years ago

I'm having hard times building mobile version of my app. In my case each user is in couple of sessions at once. One or two are open, for signalling, tracking who is online in room etc and another is private, just for video. I discovered that when on mobile device I try to connect to video session I'm automagically kicked out from the previous session. There are no docs for this plugin so I don't know if it's supposed to work this way or not.

IGitGotIt commented 9 years ago

Just tested with native Obj-C on 2.4.1 ios SDK (https://tokbox.com/opentok/libraries/client/ios/) and both sessions worked ok. I used one session to just do signaling and the other to publish, all from the same app.

Looking at https://github.com/songz/cordova-plugin-opentok/blob/master/src/ios/OpenTokPlugin.m looks like the plugin has only one session object (TB.initSession call to initSession: ). Thus if you create a second session , the first one will be released. You can maybe change the plugin to accommodate multiple sessions at the same time

Pentiado commented 9 years ago

Yeah! I was right. Thanks! So my plan for the weekend is to learn mobile native technologies to change the plugin : ) but if someone else would send pull request with those I wouldn't mind. Just saying.

aullman commented 9 years ago

@JayTokBox can you connect to multiple sessions at the same time in the OpenTok mobile SDKs? I thought that was a limitation.

sanvir10 commented 9 years ago

Me too would like use the plugin with multiples sessions i hope some can help to update the plugin...

aullman commented 9 years ago

I have confirmed that OpenTok mobile sdks do not support this. I would recommend filing an issue with TokBox supoort.

aullman commented 9 years ago

My bad, it seems that OpenTok mobile SDKs do support multiple sessions now. There was some confusion but it has been tested and apparently it works.

So the next step is to update the Cordova plugin to allow this. Any takers?