jellyfin / jellyfin-roku

The Official Roku Client for Jellyfin
https://jellyfin.org
GNU General Public License v2.0
419 stars 128 forks source link

Deleting servers on serverselect view produces unexpected results #1152

Open cewert opened 1 year ago

cewert commented 1 year ago

Software Versions

Describe the bug

There's a few:

Expected behavior

Connection Information

Additional context

Introduced in #512

The data we are saving in the registry for each saved server seems excessive

{"serverlist":[{
    "baseurl":"https://demo.jellyfin.org/stable",
    "iconheight":120,
    "iconurl":"pkg:/images/logo-icon120.jpg",
    "iconwidth":120,
    "name":"Stable Demo (Saved)",
    "password":"",
    "username":"demo"
}]}

The available_users registry key saves the same information minus the server "name" :

[{
  "id":"a076a5bfc9034f379f588XXXXXXXXX",
  "server":"https://demo.jellyfin.org/stable",
  "username":"demo"
}]
cewert commented 1 year ago

Oh and anyone who wants to test the server select view I HIGHLY recommend using RALE that way you can repopulate the saved_server registry setting by copy/pasting into RALE instead of manually doing it with the remote 😄

Here's the payload for the demo server

{"serverlist":[{"baseurl":"https://demo.jellyfin.org/stable","iconheight":120,"iconurl":"pkg:/images/logo-icon120.jpg","iconwidth":120,"name":"Stable Demo (Saved)","password":"","username":"demo"}]}