wcandillon / react-native-expo-image-cache

React Native Image Cache and Progressive Loading based on Expo
MIT License
668 stars 125 forks source link

Do not cache image results with a non-200 status result #85

Closed benburton closed 6 years ago

benburton commented 6 years ago

I've been experiencing an issue in which our CDN returns a non-200 error response for certain image assets. The response body is placed into the image cache, and when it's rendered using an <Image/>, nothing is displayed. This PR will return undefined when there is a non-200 response from an image request.

wcandillon commented 6 years ago

@benburton Thanks a lot! This is fixed in 3.2.0