Merge pull request #830 from GetStream/hotfix/media-download

fix(ui): media download
GetStream · Dec 27, 2021 · 5a4fe29 · 5a4fe29
2 parents 13a863f + 793bbba
commit 5a4fe29
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/packages/stream_chat_flutter/CHANGELOG.md b/packages/stream_chat_flutter/CHANGELOG.md
@@ -1,5 +1,9 @@
 ## Upcoming
 
+🐞 Fixed
+
+- Use file extension instead of mimeType for downloading files
+
 ✅ Added
 
 - Videos can now be auto-played in `FullScreenMedia`

diff --git a/packages/stream_chat_flutter/lib/src/attachment_actions_modal.dart b/packages/stream_chat_flutter/lib/src/attachment_actions_modal.dart
@@ -365,12 +365,13 @@ class AttachmentActionsModal extends StatelessWidget {
   }) async {
     String? filePath;
     final appDocDir = await getTemporaryDirectory();
+    final url =
+        attachment.assetUrl ?? attachment.imageUrl ?? attachment.thumbUrl!;
     await Dio().download(
-      attachment.assetUrl ?? attachment.imageUrl ?? attachment.thumbUrl!,
+      url,
       (Headers responseHeaders) {
-        final contentType = responseHeaders[Headers.contentTypeHeader]!;
-        final mimeType = contentType.first.split('/').last;
-        filePath ??= '${appDocDir.path}/${attachment.id}.$mimeType';
+        final ext = Uri.parse(url).pathSegments.last;
+        filePath ??= '${appDocDir.path}/${attachment.id}.$ext';
         return filePath!;
       },
       onReceiveProgress: progressCallback,