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

user stays permanently online #50

Open
arminfelder opened this issue Sep 5, 2019 · 4 comments
Open

user stays permanently online #50

arminfelder opened this issue Sep 5, 2019 · 4 comments

Comments

@arminfelder
Copy link

I often see users online for days, without being logged in to the system.
In the usersSession collection, there are still entries for session, which do not exist anymore...

I do experience that issue with Rocket.Chat, but I did not manage to find out, under which circumstances this happens.

@flaviogrossi
Copy link

flaviogrossi commented Sep 10, 2019

This happens to us too.
From what I've been able to debug this condition here evaluates to false and the record is never removed.

I will add more details as soon as i found out more. Anyone has any idea?

@flaviogrossi
Copy link

This happens to us too.

ok, in our case it was an old version of this package. The commit solving #31 seems to fix everything.

@rasos
Copy link

rasos commented Sep 18, 2019

Just wondering whether you needed to backport the meteor-user-presence, as this fix was already merged with RC 0.69 RocketChat/Rocket.Chat#11709

@emaciel10
Copy link

Our team also noticed this same issue and unfortunately we had to add a batch job to regularly clean up some stale connections to instances that are no longer around after a deploy, etc.

Somewhat tangentially related I also added a PR here to fix a duplicate connection issue we noticed where Accounts.onLogin was sometimes triggered more than once resulting in two connections to the same instance:
#54

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

4 participants