Update jellyfin-sdk-kotlin to v1.6.1

app/src/main/java/org/jellyfin/mobile/app/ApiClientController.kt

@@ -8,7 +8,6 @@ import org.jellyfin.mobile.data.entity.ServerEntity
 import org.jellyfin.sdk.Jellyfin
 import org.jellyfin.sdk.api.client.ApiClient
 import org.jellyfin.sdk.model.DeviceInfo
-import org.jellyfin.sdk.model.serializer.toUUID
 
 class ApiClientController(
     private val appPreferences: AppPreferences,
@@ -27,7 +26,7 @@ class ApiClientController(
         appPreferences.currentServerId = withContext(Dispatchers.IO) {
             serverDao.getServerByHostname(hostname)?.id ?: serverDao.insert(hostname)
         }
-        apiClient.baseUrl = hostname
+        apiClient.update(baseUrl = hostname)
     }
 
     suspend fun setupUser(serverId: Long, userId: String, accessToken: String) {
@@ -69,19 +68,21 @@ class ApiClientController(
     }
 
     private fun configureApiClientServer(server: ServerEntity?) {
-        apiClient.baseUrl = server?.hostname
+        apiClient.update(baseUrl = server?.hostname)
     }
 
     private fun configureApiClientUser(userId: String, accessToken: String) {
-        apiClient.userId = userId.toUUID()
-        apiClient.accessToken = accessToken
-        // Append user id to device id to ensure uniqueness across sessions
-        apiClient.deviceInfo = baseDeviceInfo.copy(id = baseDeviceInfo.id + userId)
+        apiClient.update(
+            accessToken = accessToken,
+            // Append user id to device id to ensure uniqueness across sessions
+            deviceInfo = baseDeviceInfo.copy(id = baseDeviceInfo.id + userId),
+        )
     }
 
     private fun resetApiClientUser() {
-        apiClient.userId = null
-        apiClient.accessToken = null
-        apiClient.deviceInfo = baseDeviceInfo
+        apiClient.update(
+            accessToken = null,
+            deviceInfo = baseDeviceInfo,
+        )
     }
 }

app/src/main/java/org/jellyfin/mobile/app/AppModule.kt

@@ -11,8 +11,10 @@ import com.google.android.exoplayer2.source.ProgressiveMediaSource
 import com.google.android.exoplayer2.source.SingleSampleMediaSource
 import com.google.android.exoplayer2.source.hls.HlsMediaSource
 import com.google.android.exoplayer2.upstream.DataSource
+import com.google.android.exoplayer2.upstream.DataSpec
 import com.google.android.exoplayer2.upstream.DefaultDataSource
 import com.google.android.exoplayer2.upstream.DefaultHttpDataSource
+import com.google.android.exoplayer2.upstream.ResolvingDataSource
 import com.google.android.exoplayer2.util.Util
 import kotlinx.coroutines.channels.Channel
 import okhttp3.OkHttpClient
@@ -34,6 +36,8 @@ import org.jellyfin.mobile.utils.isLowRamDevice
 import org.jellyfin.mobile.webapp.RemoteVolumeProvider
 import org.jellyfin.mobile.webapp.WebViewFragment
 import org.jellyfin.mobile.webapp.WebappFunctionChannel
+import org.jellyfin.sdk.api.client.ApiClient
+import org.jellyfin.sdk.api.client.util.AuthorizationHeaderBuilder
 import org.koin.android.ext.koin.androidApplication
 import org.koin.androidx.fragment.dsl.fragment
 import org.koin.androidx.viewmodel.dsl.viewModel
@@ -81,6 +85,7 @@ val applicationModule = module {
     // ExoPlayer factories
     single<DataSource.Factory> {
         val context: Context = get()
+        val apiClient: ApiClient = get()
 
         val provider = CronetProvider.getAllProviders(context).firstOrNull { provider: CronetProvider ->
             (provider.name == CronetProvider.PROVIDER_NAME_APP_PACKAGED) && provider.isEnabled
@@ -102,7 +107,21 @@ val applicationModule = module {
             }
         }
 
-        DefaultDataSource.Factory(context, baseDataSourceFactory)
+        val dataSourceFactory = DefaultDataSource.Factory(context, baseDataSourceFactory)
+
+        // Add authorization header. This is needed as we don't pass the
+        // access token in the url for Android Auto.
+        ResolvingDataSource.Factory(dataSourceFactory) { dataSpec: DataSpec ->
+            val authorizationHeaderString = AuthorizationHeaderBuilder.buildHeader(
+                clientName = apiClient.clientInfo.name,
+                clientVersion = apiClient.clientInfo.version,
+                deviceId = apiClient.deviceInfo.id,
+                deviceName = apiClient.deviceInfo.name,
+                accessToken = apiClient.accessToken,
+            )
+
+            dataSpec.withRequestHeaders(hashMapOf("Authorization" to authorizationHeaderString))
+        }
     }
     single<MediaSource.Factory> {
         val context: Context = get()

app/src/main/java/org/jellyfin/mobile/player/PlayerViewModel.kt

@@ -66,6 +66,7 @@ import org.jellyfin.sdk.api.operations.HlsSegmentApi
 import org.jellyfin.sdk.api.operations.PlayStateApi
 import org.jellyfin.sdk.api.operations.UserApi
 import org.jellyfin.sdk.model.api.PlayMethod
+import org.jellyfin.sdk.model.api.PlaybackOrder
 import org.jellyfin.sdk.model.api.PlaybackProgressInfo
 import org.jellyfin.sdk.model.api.PlaybackStartInfo
 import org.jellyfin.sdk.model.api.PlaybackStopInfo
@@ -319,6 +320,7 @@ class PlayerViewModel(application: Application) : AndroidViewModel(application),
                     positionTicks = mediaSource.startTimeMs * Constants.TICKS_PER_MILLISECOND,
                     volumeLevel = audioManager.getVolumeLevelPercent(),
                     repeatMode = RepeatMode.REPEAT_NONE,
+                    playbackOrder = PlaybackOrder.DEFAULT,
                 ),
             )
         } catch (e: ApiClientException) {
@@ -347,6 +349,7 @@ class PlayerViewModel(application: Application) : AndroidViewModel(application),
                         positionTicks = playbackPositionMillis * Constants.TICKS_PER_MILLISECOND,
                         volumeLevel = (currentVolume - volumeRange.first) * Constants.PERCENT_MAX / volumeRange.width,
                         repeatMode = RepeatMode.REPEAT_NONE,
+                        playbackOrder = PlaybackOrder.DEFAULT,
                     ),
                 )
             } catch (e: ApiClientException) {

app/src/main/java/org/jellyfin/mobile/player/audio/MediaService.kt

@@ -44,7 +44,6 @@ import org.jellyfin.mobile.player.cast.ICastPlayerProvider
 import org.jellyfin.mobile.utils.Constants
 import org.jellyfin.mobile.utils.extensions.mediaUri
 import org.jellyfin.mobile.utils.toast
-import org.jellyfin.sdk.api.client.ApiClient
 import org.jellyfin.sdk.api.client.exception.ApiClientException
 import org.koin.android.ext.android.get
 import org.koin.android.ext.android.inject
@@ -53,7 +52,6 @@ import com.google.android.exoplayer2.MediaItem as ExoPlayerMediaItem
 
 class MediaService : MediaBrowserServiceCompat() {
     private val apiClientController: ApiClientController by inject()
-    private val apiClient: ApiClient by inject()
     private val libraryBrowser: LibraryBrowser by inject()
 
     private val serviceScope = MainScope()
@@ -177,12 +175,7 @@ class MediaService : MediaBrowserServiceCompat() {
             loadingJob.join()
 
             val items = try {
-                if (apiClient.userId != null) {
-                    libraryBrowser.loadLibrary(parentId)
-                } else {
-                    Timber.e("Missing userId in ApiClient")
-                    null
-                }
+                libraryBrowser.loadLibrary(parentId)
             } catch (e: ApiClientException) {
                 Timber.e(e)
                 null

app/src/main/java/org/jellyfin/mobile/player/audio/car/LibraryBrowser.kt

@@ -40,9 +40,11 @@ import org.jellyfin.sdk.api.operations.UserViewsApi
 import org.jellyfin.sdk.model.api.BaseItemDto
 import org.jellyfin.sdk.model.api.BaseItemDtoQueryResult
 import org.jellyfin.sdk.model.api.BaseItemKind
+import org.jellyfin.sdk.model.api.CollectionType
 import org.jellyfin.sdk.model.api.ImageType
-import org.jellyfin.sdk.model.api.ItemFields
 import org.jellyfin.sdk.model.api.ItemFilter
+import org.jellyfin.sdk.model.api.ItemSortBy
+import org.jellyfin.sdk.model.api.MediaStreamProtocol
 import org.jellyfin.sdk.model.api.SortOrder
 import org.jellyfin.sdk.model.serializer.toUUID
 import org.jellyfin.sdk.model.serializer.toUUIDOrNull
@@ -178,10 +180,10 @@ class LibraryBrowser(
                     Timber.d("Searching for artist $artistQuery")
                     searchItems(artistQuery, BaseItemKind.MUSIC_ARTIST)
                 }?.let { artistId ->
-                    itemsApi.getItemsByUserId(
+                    itemsApi.getItems(
                         artistIds = listOf(artistId),
                         includeItemTypes = listOf(BaseItemKind.AUDIO),
-                        sortBy = listOf("Random"),
+                        sortBy = listOf(ItemSortBy.RANDOM),
                         recursive = true,
                         imageTypeLimit = 1,
                         enableImageTypes = listOf(ImageType.PRIMARY),
@@ -197,7 +199,7 @@ class LibraryBrowser(
 
         // Fallback to generic search
         Timber.d("Searching for '$searchQuery'")
-        val result by itemsApi.getItemsByUserId(
+        val result by itemsApi.getItems(
             searchTerm = searchQuery,
             includeItemTypes = listOf(BaseItemKind.AUDIO),
             recursive = true,
@@ -214,7 +216,7 @@ class LibraryBrowser(
      * Find a single specific item for the given [searchQuery] with a specific [type]
      */
     private suspend fun searchItems(searchQuery: String, type: BaseItemKind): UUID? {
-        val result by itemsApi.getItemsByUserId(
+        val result by itemsApi.getItems(
             searchTerm = searchQuery,
             includeItemTypes = listOf(type),
             recursive = true,
@@ -223,7 +225,7 @@ class LibraryBrowser(
             limit = 1,
         )
 
-        return result.items?.firstOrNull()?.id
+        return result.items.firstOrNull()?.id
     }
 
     suspend fun getDefaultRecents(): List<MediaMetadataCompat>? = getLibraries().firstOrNull()?.mediaId?.let { defaultLibrary ->
@@ -235,8 +237,8 @@ class LibraryBrowser(
     private suspend fun getLibraries(): List<MediaBrowserCompat.MediaItem> {
         val userViews by userViewsApi.getUserViews()
 
-        return userViews.items.orEmpty()
-            .filter { item -> item.collectionType.equals("music", ignoreCase = true) }
+        return userViews.items
+            .filter { item -> item.collectionType == CollectionType.MUSIC }
             .map { item ->
                 val itemImageUrl = imageApi.getItemImageUrl(
                     itemId = item.id,
@@ -286,11 +288,11 @@ class LibraryBrowser(
     }
 
     private suspend fun getRecents(libraryId: UUID): List<MediaMetadataCompat>? {
-        val result by itemsApi.getItemsByUserId(
+        val result by itemsApi.getItems(
             parentId = libraryId,
             includeItemTypes = listOf(BaseItemKind.AUDIO),
             filters = listOf(ItemFilter.IS_PLAYED),
-            sortBy = listOf("DatePlayed"),
+            sortBy = listOf(ItemSortBy.DATE_PLAYED),
             sortOrder = listOf(SortOrder.DESCENDING),
             recursive = true,
             imageTypeLimit = 1,
@@ -307,12 +309,12 @@ class LibraryBrowser(
         filterArtist: UUID? = null,
         filterGenre: UUID? = null,
     ): List<MediaBrowserCompat.MediaItem>? {
-        val result by itemsApi.getItemsByUserId(
+        val result by itemsApi.getItems(
             parentId = libraryId,
             artistIds = filterArtist?.let(::listOf),
             genreIds = filterGenre?.let(::listOf),
             includeItemTypes = listOf(BaseItemKind.MUSIC_ALBUM),
-            sortBy = listOf(ItemFields.SORT_NAME.serialName),
+            sortBy = listOf(ItemSortBy.SORT_NAME),
             recursive = true,
             imageTypeLimit = 1,
             enableImageTypes = listOf(ImageType.PRIMARY),
@@ -323,10 +325,10 @@ class LibraryBrowser(
     }
 
     private suspend fun getArtists(libraryId: UUID): List<MediaBrowserCompat.MediaItem>? {
-        val result by itemsApi.getItemsByUserId(
+        val result by itemsApi.getItems(
             parentId = libraryId,
             includeItemTypes = listOf(BaseItemKind.MUSIC_ARTIST),
-            sortBy = listOf(ItemFields.SORT_NAME.serialName),
+            sortBy = listOf(ItemSortBy.SORT_NAME),
             recursive = true,
             imageTypeLimit = 1,
             enableImageTypes = listOf(ImageType.PRIMARY),
@@ -338,7 +340,6 @@ class LibraryBrowser(
 
     private suspend fun getGenres(libraryId: UUID): List<MediaBrowserCompat.MediaItem>? {
         val result by genresApi.getGenres(
-            userId = apiClient.userId,
             parentId = libraryId,
             imageTypeLimit = 1,
             enableImageTypes = listOf(ImageType.PRIMARY),
@@ -349,10 +350,10 @@ class LibraryBrowser(
     }
 
     private suspend fun getPlaylists(libraryId: UUID): List<MediaBrowserCompat.MediaItem>? {
-        val result by itemsApi.getItemsByUserId(
+        val result by itemsApi.getItems(
             parentId = libraryId,
             includeItemTypes = listOf(BaseItemKind.PLAYLIST),
-            sortBy = listOf(ItemFields.SORT_NAME.serialName),
+            sortBy = listOf(ItemSortBy.SORT_NAME),
             recursive = true,
             imageTypeLimit = 1,
             enableImageTypes = listOf(ImageType.PRIMARY),
@@ -363,9 +364,9 @@ class LibraryBrowser(
     }
 
     private suspend fun getAlbum(albumId: UUID): List<MediaMetadataCompat>? {
-        val result by itemsApi.getItemsByUserId(
+        val result by itemsApi.getItems(
             parentId = albumId,
-            sortBy = listOf(ItemFields.SORT_NAME.serialName),
+            sortBy = listOf(ItemSortBy.SORT_NAME),
         )
 
         return result.extractItems("${LibraryPage.ALBUM}|$albumId")
@@ -404,7 +405,6 @@ class LibraryBrowser(
         if (item.type == BaseItemKind.AUDIO) {
             val uri = universalAudioApi.getUniversalAudioStreamUrl(
                 itemId = item.id,
-                userId = apiClient.userId,
                 deviceId = apiClient.deviceInfo.id,
                 maxStreamingBitrate = 140000000,
                 container = listOf(
@@ -419,11 +419,10 @@ class LibraryBrowser(
                     "wav",
                     "ogg",
                 ),
-                transcodingProtocol = "hls",
+                transcodingProtocol = MediaStreamProtocol.HLS,
                 transcodingContainer = "ts",
                 audioCodec = "aac",
                 enableRemoteMedia = true,
-                includeCredentials = true,
             )
 
             builder.setMediaUri(uri)

app/src/main/java/org/jellyfin/mobile/player/deviceprofile/DeviceProfileBuilder.kt

@@ -8,6 +8,7 @@ import org.jellyfin.sdk.model.api.ContainerProfile
 import org.jellyfin.sdk.model.api.DeviceProfile
 import org.jellyfin.sdk.model.api.DirectPlayProfile
 import org.jellyfin.sdk.model.api.DlnaProfileType
+import org.jellyfin.sdk.model.api.MediaStreamProtocol
 import org.jellyfin.sdk.model.api.SubtitleDeliveryMethod
 import org.jellyfin.sdk.model.api.SubtitleProfile
 import org.jellyfin.sdk.model.api.TranscodingProfile
@@ -72,23 +73,23 @@ class DeviceProfileBuilder(
                 container = "ts",
                 videoCodec = "h264",
                 audioCodec = "mp1,mp2,mp3,aac,ac3,eac3,dts,mlp,truehd",
-                protocol = "hls",
+                protocol = MediaStreamProtocol.HLS,
                 conditions = emptyList(),
             ),
             TranscodingProfile(
                 type = DlnaProfileType.VIDEO,
                 container = "mkv",
                 videoCodec = "h264",
                 audioCodec = AVAILABLE_AUDIO_CODECS[SUPPORTED_CONTAINER_FORMATS.indexOf("mkv")].joinToString(","),
-                protocol = "hls",
+                protocol = MediaStreamProtocol.HLS,
                 conditions = emptyList(),
             ),
             TranscodingProfile(
                 type = DlnaProfileType.AUDIO,
                 container = "mp3",
                 videoCodec = "",
                 audioCodec = "mp3",
-                protocol = "http",
+                protocol = MediaStreamProtocol.HTTP,
                 conditions = emptyList(),
             ),
         )
@@ -102,7 +103,7 @@ class DeviceProfileBuilder(
         for (i in SUPPORTED_CONTAINER_FORMATS.indices) {
             val container = SUPPORTED_CONTAINER_FORMATS[i]
             if (supportedVideoCodecs[i].isNotEmpty()) {
-                containerProfiles.add(ContainerProfile(type = DlnaProfileType.VIDEO, container = container))
+                containerProfiles.add(ContainerProfile(type = DlnaProfileType.VIDEO, container = container, conditions = emptyList()))
                 directPlayProfiles.add(
                     DirectPlayProfile(
                         type = DlnaProfileType.VIDEO,
@@ -113,7 +114,7 @@ class DeviceProfileBuilder(
                 )
             }
             if (supportedAudioCodecs[i].isNotEmpty()) {
-                containerProfiles.add(ContainerProfile(type = DlnaProfileType.AUDIO, container = container))
+                containerProfiles.add(ContainerProfile(type = DlnaProfileType.AUDIO, container = container, conditions = emptyList()))
                 directPlayProfiles.add(
                     DirectPlayProfile(
                         type = DlnaProfileType.AUDIO,
@@ -141,11 +142,6 @@ class DeviceProfileBuilder(
             maxStreamingBitrate = MAX_STREAMING_BITRATE,
             maxStaticBitrate = MAX_STATIC_BITRATE,
             musicStreamingTranscodingBitrate = MAX_MUSIC_TRANSCODING_BITRATE,
-
-            // TODO: remove redundant defaults after API/SDK is fixed
-            supportedMediaTypes = "",
-            xmlRootAttributes = emptyList(),
-            responseProfiles = emptyList(),
         )
     }
 
@@ -161,8 +157,8 @@ class DeviceProfileBuilder(
     fun getExternalPlayerProfile(): DeviceProfile = DeviceProfile(
         name = EXTERNAL_PLAYER_PROFILE_NAME,
         directPlayProfiles = listOf(
-            DirectPlayProfile(type = DlnaProfileType.VIDEO),
-            DirectPlayProfile(type = DlnaProfileType.AUDIO),
+            DirectPlayProfile(type = DlnaProfileType.VIDEO, container = ""),
+            DirectPlayProfile(type = DlnaProfileType.AUDIO, container = ""),
         ),
         transcodingProfiles = emptyList(),
         containerProfiles = emptyList(),
@@ -178,11 +174,6 @@ class DeviceProfileBuilder(
         maxStreamingBitrate = Int.MAX_VALUE,
         maxStaticBitrate = Int.MAX_VALUE,
         musicStreamingTranscodingBitrate = Int.MAX_VALUE,
-
-        // TODO: remove redundant defaults after API/SDK is fixed
-        supportedMediaTypes = "",
-        xmlRootAttributes = emptyList(),
-        responseProfiles = emptyList(),
     )
 
     companion object {