ShemrickFlannigan commented on issue #309: CB-13415 (android) Importing corrupt images using the Camera plugin c? URL: https://github.com/apache/cordova-plugin-camera/pull/309#issuecomment-361072781 @jcesarmobile @infil00p I agree that the OutOfMemoryError is a catch all that is not specific to corrupt images. But in our testing, the OutOfMemoryError is the only exception which we've found that successfully captures the corrupt images. In our testing, there is no correlation with file size. The corrupts sample image which has been provided is < 5MB. The Cordova Camera app can handle non-corrupt images which are 40MB. Here are the sample images: Corrupt 5MB image: https://www.dropbox.com/s/s7jh3eay3y23oy1/5mb_13500x13500.D2%5B1%5D.jpg?dl=0 Valid 40MB image: https://www.dropbox.com/s/cnxykbcvkbc2k1k/Airbus_Pleiades_50cm_8bit_RGB_Yogyakarta.jpg?dl=0 I would like to move forward with the generic OutOfMemoryError as a catch all exception within this Pull Request, as it solves a high impact error (corrupt images crashing the entire Cordova app). Later, if/when a more accurate exception is found, it can be applied as an enhanced exception.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
