Makussu/subtracks
1
0
Fork 0
mirror of https://github.com/austinried/subtracks.git synced 2026-11-11 07:29:14 +01:00
Code Issues Packages Projects Releases Wiki Activity

retry a read that lands past a cold transcode

Browse Source 
A transcoded stream is chunked with no length, so the app declares an upper-bound
length, and on the first request the server is still producing the stream: a seek
near that bound lands past the data and comes back as POSITION_OUT_OF_RANGE
(Navidrome answers 416 until it has cached the transcode). That first request is
what warms the cache, so give the load one retry after a second: the retry sees a
real Content-Length and the stream plays and seeks. Further out-of-range errors
are given up on rather than looped.
This commit is contained in:
austinried
2026-10-04 00:30:04 +00:00
parent 9a999498ab
commit dd6b0ea00c
3 changed files with 96 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/kotlin/com/subtracks/playback/PlaybackService.kt
+1
View File
@@ -103,6 +103,7 @@ class PlaybackService : MediaSessionService() {
.setMediaSourceFactory(
DefaultMediaSourceFactory(this)
.setEnableClippingInMediaPeriod(true)
.setLoadErrorHandlingPolicy(RetryOutOfRangePolicy())
.setDataSourceFactory(mediaDataSourceFactory(this, OkHttpDataSource.Factory(streamingClient()))),
).build()
exoPlayer.addListener(
app/src/main/kotlin/com/subtracks/playback/RetryOutOfRangePolicy.kt
+38
View File
@@ -0,0 +1,38 @@
package com.subtracks.playback
import androidx.media3.common.C
import androidx.media3.common.util.UnstableApi
import androidx.media3.datasource.DataSourceException
import androidx.media3.exoplayer.upstream.DefaultLoadErrorHandlingPolicy
import androidx.media3.exoplayer.upstream.LoadErrorHandlingPolicy
/**
* A transcoded stream is chunked with no length, so the app declares an upper-bound length, and on
* the first request the server is still producing the stream: a seek near that bound lands past the
* data and comes back as POSITION_OUT_OF_RANGE (Navidrome answers 416 until it has cached the
* transcode). That first request is what warms the cache, so one retry lands on a response with a
* real Content-Length and the stream plays and seeks properly.
*/
@UnstableApi
internal class RetryOutOfRangePolicy(
private val delayMs: Long = RETRY_DELAY_MS,
) : DefaultLoadErrorHandlingPolicy() {
override fun getRetryDelayMsFor(loadErrorInfo: LoadErrorHandlingPolicy.LoadErrorInfo): Long =
when {
loadErrorInfo.exception.isOutOfRange() -> {
if (loadErrorInfo.errorCount <= MAX_RETRIES) delayMs else C.TIME_UNSET
}
else -> {
super.getRetryDelayMsFor(loadErrorInfo)
}
}
private fun Throwable.isOutOfRange(): Boolean =
generateSequence(this as Throwable?) { it.cause }.any {
it is DataSourceException && it.reason == DataSourceException.POSITION_OUT_OF_RANGE
}
}
private const val MAX_RETRIES = 1
private const val RETRY_DELAY_MS = 1_000L