Cloudkibo / CloudKibo

CloudKibo
0 stars 0 forks source link

URGENT: CloudKibo Not working #443

Closed jekram closed 8 years ago

jekram commented 8 years ago

I am using a laptop and a desk to do testing. and I cannot get into conference on both.

"Jawaid PC" and "Jawaid Office". Jawaid Office does not have audio / Video.

Let me start with my screen on laptop. (Jawaid PC) and this is what I see:

image

jekram commented 8 years ago

Now on Jawaid Office this is what I see:

image

jekram commented 8 years ago

Now If I go to "Jawaid PC" and reload the screen then I see

image

sojharo commented 8 years ago

I am looking into it right away.

jekram commented 8 years ago

On "Jawaid Office" the screen is:

image

WE ARE PROBABLY GOING TO TWO DIFFRENT TURN SERVERS.

This needs to fixed URGENTLY.

sojharo commented 8 years ago

Yes, I have started to look into it. Looks like the handshaking problem to me. TURN doesn't decide who meets who.

It is showing jawaid pc twice. This had also once happened with us on webmeeting. This can be issue on how peers are handled during handshaking if one of them is on desktop.

I think I have understood the issue. I am pushing the correction in a moment.

jekram commented 8 years ago

I will be here for another 45 mins and then would go for Jumma and will be back. So I can test with you.

sojharo commented 8 years ago

I found the reason of why refresh was needed. I found that there was server crash at that time. First I am going to solve server crash.

jekram commented 8 years ago

Ok.

sojharo commented 8 years ago

I have solved the server crash error. I would solve the duplicate people error when power comes back. I would be able to test at 1:45. i have pushed for server crash.

sojharo commented 8 years ago

I am trying to fix this issue. The error is on part of room locking on server side when person leaves the room. With the logic of room lock, and also logic of webmeeting plus conference, the code has become little messy on socket server side.

I am going to remove the webmeeting code from logic and keeping it in a separate unused file so that we don't lose it. I will update more after solving this problem.

jekram commented 8 years ago

Why I it happening now? Under what circumstances this error shows up? In the last 2 weeks it has happened more

sojharo commented 8 years ago

I am still looking into the error. The error of showing duplicate users when one doesn't have camera or mic is produced now as this was not tested I think.

The error of server crashing might have happened before too but we could not notice as server is automatically restarted by our production tool. On server crashes, there is place where room lock logic is defined. So I am trying to understand that code too.

jekram commented 8 years ago

So it will only happen when there is no camera or mic? Not in other cases?

sojharo commented 8 years ago

The duplicate user error happens only in this case.

jekram commented 8 years ago

Ok.

Sent from Outlook Mobile

On Fri, Apr 15, 2016 at 3:08 PM -0700, "Sojharo" notifications@github.com wrote:

The duplicate user error happens only in this case.


You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/Cloudkibo/CloudKibo/issues/443#issuecomment-210664847

jekram commented 8 years ago

go home and fix it next day

sojharo commented 8 years ago

I am home and it is midnight. I would fix it in the morning. I have fixed the server crash issue. But there is one more issue which is not allowing the person to rejoin the meeting. Related to same problem. I will look more into it.

sojharo commented 8 years ago

I have solved both server crash error and other error of person rejoining the meeting. In this, now the only error left is when person with no mic joins he shows duplicated. I would also test it tomorrow fully.

jekram commented 8 years ago

Thanks - Good Night

sojharo commented 8 years ago

Person rejoining error was still there. I have just solved that error. I have pushed the code. Now I am fixing the error for person with no mic

sojharo commented 8 years ago

I have tested it with my laptop and with desktop computer here. Now the issue is not happening. The previous two problems in this issue might be causing this problem. I tested with three peers as well.

jekram commented 8 years ago

Great - I will test it later when I get back into my office.

jekram commented 8 years ago

Tested. Thanks