RaunoT / plex-rewind

Present Plex user statistics and habits in a beautiful and organized manner à la Spotify Wrapped.
GNU General Public License v3.0
69 stars 5 forks source link

Add ability to see managed user's Rewinds #37

Closed Jobinrandhawa closed 5 months ago

Jobinrandhawa commented 6 months ago

I have a login for my admin account on Plex that no one uses but I use a managed user and because I use the login of the admin account to access my managed user I cannot see the rewind for the managed user.

tarunkumar519 commented 6 months ago

This is the most important feature for alot of users. Hope this can be implemented soon

RaunoT commented 6 months ago

Open to suggestions on how to handle this.

Would we still require login in this usecase? Do we expect managed users to know the Plex login?

Here's 2 ways I can envision currently: 1) When a Plex Home admin login is used, after login there will be separate Rewind buttons for every managed user. 2) Add the ability to bypass authentication completely and have sharable links for every users Rewind.

tarunkumar519 commented 6 months ago

Plex home admin login details are used. After login there will be separate account buttons of managed users to select. But a managed user must have access to their own account and not anybody else.

yes all managed users have plex login access.

so a separate PIN based authentication method for managed users like Plex already does? Could be any method.

bypassing authentication is less work, and it could work in the meantime

RaunoT commented 6 months ago

@tarunkumar519 If the user logs in with admin login details and is showed multiple choices to pick whos Rewind to see, there is no way to ensure that "managed user must have access to their own account and not anybody else". A user using admin login details, for all intensive purposes, is the admin.

I think that if you share your admin login details with a managed users, having them have the ability to see your Rewind isn't a real added security risk.

If we bypass the whole authentication, we would need to show Rewind buttons for every user on the server, since there is no other way to know who is viewing the site.

RaunoT commented 5 months ago

:tada: This issue has been resolved in version 1.8.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: