- 🇳🇱Netherlands megachriz
It would be useful to add test coverage for this issue. Also, code now needs to be in a merge request instead of a patch because patches are no longer evaluated by the drupal.org testbot.
https://www.instagram.com/p/B9vy5EdgZX1/media/?size=l is proviuded by the Instagram API, but this actually redirects to https://scontent-lht6-1.cdninstagram.com/v/t51.2885-15/fr/e15/p1080x1080/89738651_155780038891572_5224457300248488192_n.jpg?_nc_ht=scontent-lht6-1.cdninstagram.com&_nc_cat=100&_nc_ohc=rGVubVRCFoAAX8YKjNO&oh=76c6ac182e0432bfffce7539e372631d&oe=5EB77DAB\Drupal\feeds\Feeds\Target\File::getFileName which expects to get a URI that contains a file extension that it can pass to \Drupal\Core\File\FileSystem::basename$url as part of \Drupal\feeds\Feeds\Target\File::getFileName and update the value of $url whenever a redirect takes place.
Patch incoming :)
Needs review
3.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
It would be useful to add test coverage for this issue. Also, code now needs to be in a merge request instead of a patch because patches are no longer evaluated by the drupal.org testbot.