Last modified: 2014-10-16 11:32:09 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T73066, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 71066 - Users are not logged out when logging in as another user
Users are not logged out when logging in as another user
Status: NEW
Product: MediaWiki
Classification: Unclassified
User login and signup (Other open bugs)
unspecified
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-19 20:04 UTC by Tyler Romeo
Modified: 2014-10-16 11:32 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Tyler Romeo 2014-09-19 20:04:51 UTC
When already logged in, the login form (after I7e40c13a6ca5) will allow users to log in again as another user. However, in this scenario, LoginForm does not call $user->logout() on the original user.

The question: should it?

Extensions may be expecting the appropriate hooks to be called any time a user ends their session (and in this case, even though they are switching users, it can be considered an end to the session).

For example, Extension:SecureSessions keeps track of where a user is logged in, and shows that information to the user. However, it depends on the UserLogout hook to know when a session is terminated.
Comment 1 Bartosz Dziewoński 2014-09-19 21:18:59 UTC
(In reply to Tyler Romeo from comment #0)
> Extensions may be expecting the appropriate hooks to be called any time a
> user ends their session (and in this case, even though they are switching
> users, it can be considered an end to the session).
> 
> For example, Extension:SecureSessions keeps track of where a user is logged
> in, and shows that information to the user. However, it depends on the
> UserLogout hook to know when a session is terminated.

I don't see how this can possibly work when a user can just clear their cookies? I must be missing something.
Comment 2 Tyler Romeo 2014-10-16 11:32:09 UTC
Somehow didn't see the response.

Yes it is not entirely accurate. However, think about the workflow for this. If somebody closes out there browser on another computer and the cookies are cleared for whatever reason, the user will probably not be surprised to see that session still listed.

But if a user explicitly logs off of a computer (or in this scenario, logs in as another user and then logs out), and then goes later to see that session still listed in their list of sessions, it will definitely be confusing and they might wonder whether they were actually logged out or not.

That is why I think it would be best to, at the very least, try and make the session list as accurate as it can possibly be.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links