CacheManager attempts to create file named guid.* on Windows (full example included and clear description)
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 38/100
Piste de recherche
La logique suspectée se trouve dans lib/src/web/file_service.dart, vers la ligne 115 ; commencez par exécuter la reproduction minimale fournie sous Windows et suivez la manière dont l’en-tête content-type devient l’extension du nom de fichier du cache. Le travail est terminé lorsque les téléchargements dont les en-têtes ne fournissent aucune extension utilisable créent un nom de fichier valide sous Windows, tandis que les extensions normales continuent de fonctionner.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
🐛 Bug Report
While downloading an image from a webserver the cache manager decides to create a filename ending with .* which is not permitted on windows.
Expected behavior
Create a filename without extension or a generic extension like .cache or .file since the cache manager cannot determine it from the webserver headers.
Reproduction steps
i have made minimal reproducible example no external dependencies needed (other than http package)
i am aware that the server should provide correct descriptive headers for the content being served but sometimes it happens not to in some scenarios.
import 'package:flutter_cache_manager/flutter_cache_manager.dart';
import 'package:http/http.dart' as http;
import 'package:http/testing.dart';
Future<void> main() async {
final client = MockClient((http.Request request) async =>
http.Response("a png image", 200, headers: {
"content-type": "application/*",
"content-disposition":
"attachment; filename=file; filename*=UTF-8''file"
}));
final cmgr = CacheManager(Config(DefaultCacheManager.key,
fileService: HttpFileService(httpClient: client)));
/* crashes with
Launching lib\main.dart on Windows in debug mode...
√ Built build\windows\runner\Debug\cache_mgr_test.exe.
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: FileSystemException: Cannot open file, path = 'C:\Users\csman\AppData\Local\Temp\libCachedImageData\cf09f070-6cef-1d8a-b096-2334712b3134.*' (OS Error: The filename, directory name, or volume label syntax is incorrect.
, errno = 123)
#0 _checkForErrorResponse (dart:io/common.dart:55:9)
#1 _File.open.<anonymous closure> (dart:io/file_impl.dart:381:7)
<asynchronous suspension>
*/
await cmgr.downloadFile("http://myaddress.com/imageManager?pngfileid=1234");
}
/*
pubspec.yaml
name: cache_mgr_test
description: A new Flutter project.
publish_to: "none"
version: 1.0.0+1
environment:
sdk: ">=3.1.5 <4.0.0"
dependencies:
cupertino_icons: ^1.0.2
flutter:
sdk: flutter
flutter_cache_manager: ^3.3.1
http: ^1.1.0
dev_dependencies:
flutter_lints: ^2.0.0
flutter_test:
sdk: flutter
flutter:
uses-material-design: true
*/
Configuration
Version: 3.3.1
Platform:
Windows 10
I think the problematic part is in \flutter_cache_manager-3.3.1\lib\src\web\file_service.dart line 115 around
@override
String get fileExtension {
var fileExtension = '';
final contentTypeHeader = _header(HttpHeaders.contentTypeHeader);
if (contentTypeHeader != null) {
final contentType = ContentType.parse(contentTypeHeader);
fileExtension = contentType.fileExtension;
}
return fileExtension;
}
- Langage dominant
- Dart
- Étoiles
- 810
- Forks
- 510
- Merge moyen
- 2 j 2 h
- PR mergées (30 j)
- 11
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Baseflow/flutter_cache_manager
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Baseflow/flutter_cache_manager#450 · 9 commentaires · 8 réactions ·
-
Retaining original file names Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 48/100
Baseflow/flutter_cache_manager#507 · 1 commentaire ·
-
sqflite_android update Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 30/100
Baseflow/flutter_cache_manager#501 · 1 commentaire · 3 réactions ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
Baseflow/flutter_cache_manager#500 · 1 commentaire · 3 réactions ·
Toutes les issues de Baseflow/flutter_cache_manager
Issues similaires
-
status: needs-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
a2ui-project/a2ui#2782 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
MaikuB/flutter_local_notifications#2832 · 1 commentaire ·
-
[Bug]: New version of nm package Ouvertebug triage
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
fluttercommunity/plus_plugins#3979 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
from.page-issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100