Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bugbounty: $400] Possible regression of #21004 / Call already ended #21055

Closed
fdellwing opened this issue Mar 10, 2021 · 34 comments
Closed

[Bugbounty: $400] Possible regression of #21004 / Call already ended #21055

fdellwing opened this issue Mar 10, 2021 · 34 comments

Comments

@fdellwing
Copy link

fdellwing commented Mar 10, 2021

$400 bounty for solving this issue: https://www.bountysource.com/issues/97216131-possible-regression-of-21004-call-already-ended


Description:

After updating from 3.12.0 to 3.12.1 we get "call already ended" a few seconds after someone started a call. This might be a regression of #21004.

Steps to reproduce:

  1. Start a new call in a new windows
  2. Wait 10sec
  3. Click on "click to join" with another user

Expected behavior:

The running call should open.

Actual behavior:

"Call already ended"

Server Setup Information:

  • Version of Rocket.Chat Server: 3.12.1
  • Operating System: Ubuntu 18.04
  • Deployment Method: docker-compose
  • Number of Running Instances: 3
  • DB Replicaset Oplog: enabled
  • NodeJS Version: 12.18.4
  • MongoDB Version: 3.6.9

Client Setup Information

  • Desktop App or Browser Version: Linux App/Chrome
  • Operating System: Kubuntu/Win10

Additional context

We use a self hosted Jitsi instance with Ubuntu 18.04 and Jitsi Meet 2.0.5390-3.

Relevant logs:

None

@KodoGit
Copy link

KodoGit commented Mar 10, 2021

Same with snap version.

@klepptor
Copy link

Same here with docker RC and native Jitsi Meet.

@bognerf
Copy link

bognerf commented Mar 11, 2021

Same here, after updating from 3.11.x to 3.12.1 running on Ubuntu 18.04 we experience identical issues as thread starter does.

@yash-rajpal
Copy link
Member

@fdellwing and all others suffering from this, I am very sorry that closing tab bar after initiating a call is having such serious problems, and I am looking into it. I tested this but didn't tested for this issue or thought this could be a possible regression of #21004
That said, in earlier versions the tab bar used to automatically close after connecting the call and this issue wasn't there. Also I think there is a much bigger underlying issue, the mentioned pr just closes tab bar automatically, but what if user decides to close tab manually, after initiating call, it would still have the same effect. I will try to fix this asap. :)

@fdellwing
Copy link
Author

Yeah, initially the issue is surely to be found in the react rewrite of the component. But closing the sidebar has an impact on it, so I mentioned it here. Thanks @yash-rajpal.

@HiveMindNet
Copy link

Anyone know of a temp fix we could implement while we wait for a proper fix in a pull please?

@fdellwing
Copy link
Author

Just click on the call symbol again. It's a bit of spam in a large group, but works perfectly fine.

@HiveMindNet
Copy link

HiveMindNet commented Mar 17, 2021

Just click on the call symbol again. It's a bit of spam in a large group, but works perfectly fine.

Thanks - That's what we've been doing. Actually, if it helps anyone else, we created this which we need to keep posting.

Trouble Joining Video Call - READ THIS

I was more hoping for some code to tweak though in the meantime, so that we can quick-fix before the real fix is in a pull.

Thanks :)

@KodoGit
Copy link

KodoGit commented Mar 19, 2021

If RC uses the same URL every time we press "call" button than I miss the purpose of all this "call already ended".
Shouldn't it generate different room URL every time?
Now if some one manage to get a URL he can join all the meetings...

@fdellwing
Copy link
Author

Congratulations, you understood how Jitsi works. If you have the link you can join. That is the reason you setup authentication.

@KodoGit
Copy link

KodoGit commented Mar 19, 2021

I know how Jitsi works. I thought that RC generates different URL over time. But now it is the same url for RC channel for days.

@arpsyapathy
Copy link

Hi all!

I have the same problems after upgrade!

@ankar84
Copy link

ankar84 commented Apr 5, 2021

It could be related to jitsi:updateTimeout method that exist in that Call contextual bar and now it's not executed because Call contextual bar is closed right after call is started.
But anyway that fixes massive Click to Join! buttons creation issue described here RocketChat/Rocket.Chat.Electron#1847 (comment)

@emikolajczak
Copy link

Issue still exists in 3.13.0.

@emikolajczak
Copy link

emikolajczak commented Apr 8, 2021

Hi @sampaiodiego , there is any chance that this Jitsi bug will be fixed in 3.13.1?
Broken Jitsi blocks upgrade to latest versions. Working Jitsi is crucial for users.

@emikolajczak
Copy link

Hi, same issue in 3.13.1
@yash-rajpal could you tried to fix this bug?
It looks like Jitsi integration is abandoned. Sad, it is very important and key functionality for most of users.

@fdellwing
Copy link
Author

fdellwing commented Apr 9, 2021

@emikolajczak You are aware of the fact, that Jitsi works fine in 3.12.x and 3.13.x?

Yes the "call already ended" is annoying, but not a big problem. So it is absolutly logical, that it has not that high of a prio.

@emikolajczak
Copy link

In my opinion it not fine. It will be very hard to explain my users that in previous version they had almost good solution, and now they have to start call again when need to join.

@fdellwing
Copy link
Author

It's ok if that is your opinion, you have three possibilities:

  1. Fix it yourself
  2. Pay someone to fix it for you
  3. Wait until someone fixes it

But please stop posting here after every update that it still does not work. If it is fixed, this issue will be closed (and the bugfix will be present in the changelogs). If this issue is not closed, it is not fixed. No need to spam (currently) 8 persons with an absolut useless information.

@pawcykca
Copy link

pawcykca commented Apr 9, 2021

@fdellwing

"Yes the "call already ended" is annoying, but not a big problem. So it is absolutly logical, that it has not that high of a prio."

This is only your opinion, because you probably don't use JITSI integration on a daily basis. If someone is actively reporting a problem on a thread, rather mean that it is a big problem for him.
Discouraging someone from reporting bugs is not a professional approach, so it's worth considering.

@fdellwing
Copy link
Author

This is only your opinion, because you probably don't use JITSI integration on a daily basis.

We use Jitsi in our company very activly and it is not a big problem. It was literally one message describing the problem and how to work around it.

Discouraging someone from reporting bugs is not a professional approach, so it's worth considering.

The bug already got reported, otherwise this issue would not exist. Writing 3 posts in less than 2 days without adding any meaningfull information to the topic is "not a professional approach".

@sandrotanner
Copy link

sandrotanner commented Apr 22, 2021

@yash-rajpal Were you able to find the time to work on this issue? Or do you have any hints that might help someone else to work on it?

@HiveMindNet
Copy link

Does anyone know if this workaround works? I tried but it didn't for me but I couldn't restart the server as people were using it.

#11649 (comment)

If it does after a restart, I'll apply it after hours.

Thinking that this "it is not a big problem" is a UX joke. We don't mind paying for fixes if there's a way to do it that doesn't involve buying an enterpriser license. This has surely gone on for too long.

Is there no way to pay for expedited fixes that benefit everyone anyway??

@fdellwing
Copy link
Author

You can always post a bounty here: https://www.bountysource.com/

Sadly the RC developers itself do not care much about Jitsi.

@HiveMindNet
Copy link

$400 Bounty posted.

https://www.bountysource.com/issues/97216131-possible-regression-of-21004-call-already-ended

@HiveMindNet
Copy link

Apparently I need to be a 'public member of the GitHub organization to enable the plugin' I need to highlight that I've posted a bounty on this issue. Anyone know how I do that? 🤣🤣🤣

@fdellwing fdellwing changed the title Possible regression of #21004 / Call already ended [Bugbounty: $400] Possible regression of #21004 / Call already ended Apr 26, 2021
@fdellwing
Copy link
Author

I can only edit it into the issue itself, but that should make it at least a bit more visible.

@yash-rajpal
Copy link
Member

@fdellwing @HiveMindNet I have implemented the basic solution. Please have a look at it and test it, also please let me know about this. :)

Please reach out to me at open rocket chat and we can discuss and review solution.

Thanks :)

@HiveMindNet
Copy link

Hi @yash-rajpal - I'm running a docker based instance. Until that's pushed I guess I can test by just replacing the files you worked on? I've dropped you a note as you suggest on open rocket chat and maybe you can point me at files I need to replace?

Happy that everyone using Jitsi benefits from this :)

Thanks

@HiveMindNet
Copy link

@fdellwing - @yash-rajpal has fixed the issue as far as I'm concerned so it's fair we should give him the bounty - however as I added the bounty on an issue you created, you need to please close the issue. He has also already posted his solution. As it's your issue, I can't close it and release the payment to him. I will also email the guys at bounty source an ask them to release the payment.

If you wouldn't mind - I'd also like to cc you on the email I'm sending the about guys so that you can confirm to them. If you're ok with that, please share your email with me here https://open.rocket.chat/direct/HiveMindNet

Thanks very much @yash-rajpal

@fdellwing
Copy link
Author

Sure I'll close the issue. Thank you @yash-rajpal for solving the issue. Lets hope it gets merged into RC soon.

@yash-rajpal
Copy link
Member

@HiveMindNet @fdellwing Thank You very much. :)

@HiveMindNet is already running the fix on his self hosted Rocket Chat server and it works for him.

If anyone wants the fix for this urgently for their server, you can reach out to me at - https://open.rocket.chat/direct/yash.rajpal - and I can help you with the fix ( by providing a docker image for the fix or some other workaround ).

Hoping the PR gets reviewed soon. :)

@pjsft
Copy link

pjsft commented May 20, 2021

@yash-rajpal @sampaiodiego Do you have any amendment deadline? The problem occurs for nearly 3 months. :/

@ankar84
Copy link

ankar84 commented May 21, 2021

@yash-rajpal @sampaiodiego Do you have any amendment deadline? The problem occurs for nearly 3 months. :/

#21808 merged today and will be available in 3.15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests