More error tolerant attempt to save /fetch remote image #231
Comments
|
Would a retry mechanism help to mitigate the issue? I would like to still fail the process if one remote image does not load. (People running this in CI would complain otherwise) |
|
Yeah that might just work! - I'll chuck your PR on our preprod site CI where i can reliably hit the issue when I get a chance and will try to get back to you hopefully in a week or so. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Heya! one more quick potential change from me to gather thoughts.
Im using this tool to use SSG by conecting to contentful and downloading all the remote images. Now sometimes contentful fails to provide the image. Would it be possible to add a mode to
downloadImagesInBatchesin which theUnable to save ${formattedFilename} (empty file).error (and maybe other errors). Resolve() out rather than rejection.Just as otherwise i'd have to query every image first / it seems a shame for the whole process to die out because of one image-download failure.
Hope the above makes sense, again if you okay it im happy to give it a go to make the change. Thanks,
The text was updated successfully, but these errors were encountered: