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 changing password disconnects other accounts. #17035

Closed
cooljacob204 opened this issue Sep 6, 2019 · 4 comments · Fixed by #17194
Closed

User changing password disconnects other accounts. #17035

cooljacob204 opened this issue Sep 6, 2019 · 4 comments · Fixed by #17194
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug needs info stale Ticket or PR with no recent activity

Comments

@cooljacob204
Copy link

cooljacob204 commented Sep 6, 2019

Steps to reproduce

  1. Two users logged in
  2. One user changes their password
  3. Other user gets disconnected

Expected behaviour

No one should get disconnected

Actual behaviour

Other user will often get disconnected. A quick test I do is open the admin monitoring app which polls the server often. Then when I have a user change their password the admin get disconnected.

Server configuration

Happens on a fresh snap/docker install. Tested with snap on ubuntu and centos and on the official docker image.

Operating system:
Centos, Ubuntu

Web server:
Apache

Database:
Mysql3, sqlite

PHP version:
Whatever ships with latest docker & snap versions.

Nextcloud version: (see Nextcloud admin page)
16.0.4

Updated from an older Nextcloud/ownCloud or fresh install:
Fresh

Where did you install Nextcloud from:
Docker / Snap

Other

Forgive me for deleting the rest of the sections that you are asking for. It's literally a fresh run of the snap/docker image that is failing and I am not seeing any errors. From what I can tell, it is dumping sessions.

docker run -d -p 8081:80 nextcloud

My docker daemon is running on Ubuntu 18.04. Tested snap on Centos and Ubuntu.

Thanks!

@cooljacob204 cooljacob204 added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Sep 6, 2019
@rullzer
Copy link
Member

rullzer commented Sep 9, 2019

I assume they are connected on the same account.
Because then it is expected. When you change your passwords the apptokens are updated. But all other active websessions are killed.

@cooljacob204
Copy link
Author

cooljacob204 commented Sep 9, 2019

Users are on their own separate accounts.

When user1 changes the password user2, user3 and user4 will be booted out if they attempt to change passwords or do any actions for a little bit.

For example:

Kate is online and looking at the monitoring page. Bob then changes their password on a separate device. Bob's session remains and is fine but Kate is disconnects and must log in again. Other users also usually get booted off.

Out of 5-6 users 4 will usually get booted off if one user updates their password. It isn't a consistent number between tests. But I have found that if you are logged into an admin account and on a page that poles then you will almost always get booted off.

@ghost
Copy link

ghost commented Oct 9, 2019

This issue has been automatically marked as stale because it has not had recent activity and seems to be missing some essential information. It will be closed if no further activity occurs. Thank you for your contributions.

@ghost ghost added the stale Ticket or PR with no recent activity label Oct 9, 2019
@kesselb
Copy link
Contributor

kesselb commented Oct 9, 2019

cc @skjnldsv the bot is going wild 🙈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug needs info stale Ticket or PR with no recent activity
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants