diff --git a/lib/services/download_manager/download_isolate_pool.dart b/lib/services/download_manager/download_isolate_pool.dart index 4c57da76..f06a9320 100644 --- a/lib/services/download_manager/download_isolate_pool.dart +++ b/lib/services/download_manager/download_isolate_pool.dart @@ -509,9 +509,14 @@ Future _downloadFile( var request = Request('GET', Uri.parse(pageUrl.url)); request.headers.addAll(pageUrl.headers ?? {}); StreamedResponse response = await client.send(request); - if (response.statusCode != 200) { + // Accept any 2xx — including 206 Partial Content, which the server + // returns when the source extension sends `Range: bytes=0-` on the + // streaming request (e.g. AnimeGG). Rejecting 206 here caused 3 + // retries → silent stall. + if (response.statusCode < 200 || response.statusCode >= 300) { throw DownloadPoolException( - 'Failed to download file: ${pageUrl.fileName!}', + 'Failed to download file: ${pageUrl.fileName!} ' + '(status ${response.statusCode})', ); } int total = response.contentLength ?? 0; @@ -617,8 +622,12 @@ Future _downloadSegment( } StreamedResponse response = await _withRetry(() => client.send(request), 3); - if (response.statusCode != 200) { - throw DownloadPoolException('Failed to download segment: ${ts.name}'); + // Accept any 2xx (including 206 Partial Content) — see comment in + // _downloadFile. + if (response.statusCode < 200 || response.statusCode >= 300) { + throw DownloadPoolException( + 'Failed to download segment: ${ts.name} (status ${response.statusCode})', + ); } final sink = file.openWrite(); diff --git a/lib/utils/extensions/string_extensions.dart b/lib/utils/extensions/string_extensions.dart index c3f4dd45..0b0e4aeb 100644 --- a/lib/utils/extensions/string_extensions.dart +++ b/lib/utils/extensions/string_extensions.dart @@ -59,7 +59,11 @@ extension StringExtensions on String { } bool isMediaVideo() { - return [ + // Match against the URL path only — query strings (e.g. AnimeGG's + // `?for=...`) and fragments must not defeat the suffix check. Use a + // leading `.` so e.g. `flashmp4` doesn't accidentally match. + final lower = (Uri.tryParse(this)?.path ?? this).toLowerCase(); + return const [ "3gp", "avi", "mpg", @@ -73,7 +77,7 @@ extension StringExtensions on String { "wmv", "mkv", "mov", - ].any((extension) => toLowerCase().endsWith(extension)); + ].any((extension) => lower.endsWith(".$extension")); } }