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

Indicate which users have an active session and show their client ip #2540

Closed
kroepke opened this issue Jul 26, 2016 · 3 comments
Closed

Indicate which users have an active session and show their client ip #2540

kroepke opened this issue Jul 26, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@kroepke
Copy link
Member

@kroepke kroepke commented Jul 26, 2016

To pinpoint users which cause expensive traffic or otherwise impact stability, show which users are active (i.e. have a non-expired session), their last activity and their client IP address in the user list, if available.

@kroepke kroepke added the feature label Jul 26, 2016
@kroepke kroepke added this to the 2.1.0 milestone Jul 26, 2016
@kroepke kroepke self-assigned this Jul 26, 2016
kroepke added a commit that referenced this issue Jul 26, 2016
 - use better overflow when users have many roles, fixes #2526
 - ignore non-updatable properties in ChangeUserRequest fixes #2528
 - show session information in the user list fixes #2540
@gruselglatz
Copy link

@gruselglatz gruselglatz commented Jul 27, 2016

Maybe consider adding a "kick-out" or "close session" feature?

@kroepke
Copy link
Member Author

@kroepke kroepke commented Jul 27, 2016

It's pretty late in the release cycle (even to add this change) but I'll bring it up with the team.
Originally we wanted to build a more complete interface for each of the authenticators, including a session management page, but simply ran out of time.

@kroepke
Copy link
Member Author

@kroepke kroepke commented Jul 27, 2016

We've had a discussion offline and the majority was against pulling it into the release this late. I'll move it to a future release and open a separate issue for this.

Thanks!

@kroepke kroepke mentioned this issue Jul 27, 2016
0 of 3 tasks complete
edmundoa added a commit that referenced this issue Aug 1, 2016
#2541)

* display session information, fix usability in user list, editing users

 - use better overflow when users have many roles, fixes #2526
 - ignore non-updatable properties in ChangeUserRequest fixes #2528
 - show session information in the user list fixes #2540

* move inline styles into separate component css file

* check for permissions when rendering edit user form

in order to allow reader users to change their details we need to avoid redirecting to a page they cannot see
also add new permission for changing authentication settings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants
You can’t perform that action at this time.