save-in icon indicating copy to clipboard operation
save-in copied to clipboard

not saving images whose URLs have dynamic parameter suffixes

Open qlassiqa opened this issue 5 years ago • 2 comments

The extension doesn't save images whose URLs contain some dynamic parameters suffixed to the image filename. For example, URLs like cdnb.artstation.com/..../mei-mo-af-small.jpg?1530540305 will simply throw a 'Failed - Download Error' when trying to save, then it opens the Chrome save dialog to manually save it, making it an unfriendly and slow user experience.

Another bug is related to how Squarespace-based website images are saved, which have URLs like images.squarespace-cdn.com/..../Squarespace+Coffee+Shop+Website+Example?format=1500w - in this case, the extension saves the lowest resolution image instead of the one displayed at 1500px (I assume the extension just removes the parameter and thus saves the low resolution image instead of the one actually displayed to the user with that parameter, so you basically end up saving only low quality images instead of the ones shown to you)

Note: the above bugs happen in both Chrome & Firefox on Windows.

qlassiqa avatar Dec 18 '19 10:12 qlassiqa

This shouldn't be happening as it used to save twitter images with params at the end. Needs investigation.

gyng avatar Dec 20 '19 02:12 gyng

FWIW, I just tested the extension (Chrome, Windows) with both URLs and it worked fine.

edjroot avatar May 20 '20 18:05 edjroot