Avatar from MyProfile is being downloaded without an extension (.jpg, etc)

[flutterVlad]

Summary

Когда в профиле загружена аватарка, при её скачивании из галереи скачивается файл без расширения. При проверке обнаружилось, что при скачивании в name передаётся checksum, и файл имеет имя checksum без расширения.

Steps to reproduce

1. Установить аватар на странице профиля.
2. Зайти в галерею аватара.
3. Скачать изображение.
4. Наблюдать, что файл скачен, но без расширения и определяется как файл, а не изображение.

What is the current bug behavior?

Скачивается файл без расширения.

What is the expected correct behavior?

Скачивается изображение с расширением.

[Alienjob]

Имя изображения устанавливается в файле big_avatar.dart

216 GalleryItem.image(
217      avatar!.original.url, // 'https://messen ... riginal/.jpg'
218      avatar.original.checksum ?? DateTime.now().toString(), // имя изображения

Для корректного сохранения изображения можно получить расширение из url
Дату лучше отформатировать чтобы в имя файла не попали запрещенные символы
Либо имя файла можно полностью формировать из url, отбросив домен
'https://messenger.soc.stg.t11913.org/files/public/4c/5d/97/26/a2/40/41/21/9b/78/76/6e/49/46/97/5e/original/.jpg' => 4c5d9726a24041219b78766e4946975e.jpg
Формирование имени файла можно вынести в platform_utils.dart -> getImageFileName(ImageFile)

Затем в platform_utils.dart -> saveToGallery обрабатывает эти значения. Эта функция тоже имеет доступ к url и name и могла бы подставлять расширение, но она ожидает на вход корректное имя. Она так-же проверяет что расширение не svg.

Есть опасность что не только аватар передает в platform_utils.dart -> saveToGallery некорректные данные. Можно добавить assert что имя переданного файла содержит расширение и вызов getImageFileName(ImageFile) если имя не содержит расширения

Есть опасность что в url не будет расширения. Тогда для определения типа по содержимому файла воспользоваться util -> mime -> MimeResolver.

[SleepySquash]

@Alienjob,

Есть опасность что в url не будет расширения

Да, вполне возможно. Идея с определением mime типа лично мне нравится, звучит логично.

Если DateTime.now().toString() может содержать запрещённые символы, то логично эти символы тут отгрызать, согласен.