It would be better to use retryclient.requests and a list of requests to have different fallbacks when fetching an image.
The order should be:
collections API MediaUrl
async details API contentUrl
collections API item['content']['thumbnails'][0]['thumbnailUrl']
async details API thumbnailUrl
However to prevent unnecessary requests, the main image and thumbnail url should be compared first, as they are often the same.
It would also be nice to see some kind of statistic to see how many images failed to download, as the current implementation doesn't account for it and the output may be misleading.
Moved to #26.
It would be better to use retryclient.requests and a list of requests to have different fallbacks when fetching an image.
The order should be:
MediaUrl
contentUrl
item['content']['thumbnails'][0]['thumbnailUrl']
thumbnailUrl
However to prevent unnecessary requests, the main image and thumbnail url should be compared first, as they are often the same.