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

Heal "In a call" user status #13292

Open
nickvergessen opened this issue Sep 13, 2024 · 0 comments
Open

Heal "In a call" user status #13292

nickvergessen opened this issue Sep 13, 2024 · 0 comments
Labels
1. to develop bug feature: api 🛠️ OCS API for conversations, chats and participants feature: call 📹 Voice and video calls feature: integration 📦 Integration with 3rd party (chat) service

Comments

@nickvergessen
Copy link
Member

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

It's artificially possible to end up with a stuck "In a call" user status.

Steps to reproduce

diff --git a/lib/Status/Listener.php b/lib/Status/Listener.php
index a22cc8236..c40a020e1 100644
--- a/lib/Status/Listener.php
+++ b/lib/Status/Listener.php
@@ -80,6 +80,7 @@ class Listener implements IEventListener {
                        }
                }
 
+               sleep(5);
                $this->statusManager->setUserStatus(
                        $userId,
                        'call',
  1. Apply patch to delay setting the actual user status to "In a call", but it could happen e.g. when the start of a call takes long due to federation or many push notifications, etc.
  2. Start a call and quickly leave it again
  3. Wait 5 seconds
  4. Check your user status

Expected behaviour

Your user status returns to Online

Actual behaviour

Your user status is "In a call" for the rest of the time

Possible idea

When fetching the room list, check if there is a call in any of your rooms. If No and your user status is "In a call" (message_id=call, as the message text is translated) we should reset it. Sadly the message id is not part of the public API.

@nickvergessen nickvergessen added this to the 💙 Next Patch (30) milestone Sep 13, 2024
@nickvergessen nickvergessen added the feature: api 🛠️ OCS API for conversations, chats and participants label Sep 18, 2024
@nickvergessen nickvergessen added feature: integration 📦 Integration with 3rd party (chat) service feature: call 📹 Voice and video calls labels Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop bug feature: api 🛠️ OCS API for conversations, chats and participants feature: call 📹 Voice and video calls feature: integration 📦 Integration with 3rd party (chat) service
Projects
None yet
Development

No branches or pull requests

1 participant