mirror of
https://github.com/JustinZeus/Wholphin.git
synced 2026-07-08 15:50:16 +02:00
Fix Seerr Local/Jellyfin authentication and request permissions (#734)
## Description When using LOCAL or JELLYFIN authentication methods to connect to Seerr, the login flow was not properly establishing session cookies and user permissions. The login would 'succeed' and you were led to thinking it worked, but then... This caused: - Request functionality to be greyed out even though the user should have access - Users to appear to have no permissions after login (permissions=0) ## Solution This PR fixes the authentication flow by: **Fetching full user config after login**: For LOCAL and JELLYFIN auth methods, the login endpoints (`authLocalPost()` and `authJellyfinPost()`) establish the session cookie but return incomplete user data. This PR now calls `authMeGet()` immediately after login to fetch the fully-populated user configuration including proper permissions. ## Tests - Have working apk on my dev fork. - Ran logins as Local + API. Cannot test Jellyfin due to environment, but codewise, it looks correct. ## Related issues Related to https://github.com/damontecres/Wholphin/issues/747 --------- Co-authored-by: Ray <154766448+damontecres@users.noreply.github.com>
This commit is contained in:
parent
3ada35c731
commit
1bffa413a2
1 changed files with 2 additions and 0 deletions
|
|
@ -196,6 +196,7 @@ private suspend fun login(
|
|||
password = password ?: "",
|
||||
),
|
||||
)
|
||||
client.usersApi.authMeGet()
|
||||
}
|
||||
|
||||
SeerrAuthMethod.JELLYFIN -> {
|
||||
|
|
@ -205,6 +206,7 @@ private suspend fun login(
|
|||
password = password ?: "",
|
||||
),
|
||||
)
|
||||
client.usersApi.authMeGet()
|
||||
}
|
||||
|
||||
SeerrAuthMethod.API_KEY -> {
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue