Baseflow / flutter_cached_network_image

Download, cache and show images in a flutter app
https://baseflow.com
2.44k stars 654 forks source link

If loading an image fails with maxHeightDiskCache specified, errorWidget will not be called next time. #980

Open YuyaKoshimizu opened 2 weeks ago

YuyaKoshimizu commented 2 weeks ago

🐛 Bug Report

As the title says, if CachedNetworkImage with maxHeightDiskCache (or maxWidthDiskCache) fails to load an image (403, etc.), errorWidget is called the first time, but not the next time. I think it is probably due to the fact that the image that failed to load is cached, but is there a solution? I would like to add that I have tried deleting the cache, but it is not possible to do so. Thanks for the answer.

Expected behavior

Images that fail to load are not cached.

Reproduction steps

return CachedNetworkImage(
  imageUrl: path, // Path of the image that results in a 403 error
  width: 128,
  height: 128,
  maxWidthDiskCache: 128,
  maxHeightDiskCache: 128,
  errorWidget: (context, url, error) {
    // First time through, but after that it goes through.
    return const SizedBox.shrink();
  },
  placeholder: (context, url) {
    // errorWidget is not called, so it is here to stay
    return const CircularProgressIndicator();
  },
  imageBuilder: (context, imageProvider) {
    // omission
  },
  errorListener: (value) async {
    // It tries to delete the cache, but it is always false.
    final result = CachedNetworkImage.evictFromCache(path);
    print('delete cache: $result'); // delete cache: false
  },
);

Configuration

Version:

Platform:

Sovann72 commented 3 days ago

Same problem here.

Mine happens in

Edit: If I don't specify both maxHeightDiskCache and maxWidthDiskCache values to CacheNetworkImage, it works fine as expected.