ngovil21 / RequestChannel.bundle

Request Channel - A Plex Channel to create requests
158 stars 22 forks source link

All users showing up as "Admin" #26

Closed jakeman2048 closed 8 years ago

jakeman2048 commented 8 years ago

It seems checkAdmin() always returns true. I think this is because /myplex/account doesn't require any authentication to be viewed, so adding a non-admin user X-Plex-Token header won't make the page fail to load outright. I think a possible fix would be to check if user's token matches with the token returned by /myplex/account?

I'm not using Plex Home. I'm only sharing with friends, so this behavior might be different.

jakeman2048 commented 8 years ago

oops, I forgot to check "Authenticate admin user through Plex.tv (Disable if you would like to check locally" to be compatible with Plex.TV users.