Make WordPress Core

Opened 4 years ago

Last modified 3 weeks ago

#18730 new defect (bug)

media_sideload_image() doesn't handle dynamic image url's well

Reported by: dd32 Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 2.9
Component: Media Keywords: has-patch needs-refresh
Focuses: Cc:


Example URL: http://www.tumblr.com/photo/1280/10464566223/1/tumblr_lrum2xzkpC1r3z8e3

As a result, the preg_match() for the file extension fails and issues a notice (no file extension in that url, although the redirection location does have).

The handling of a download failure also seems hairy, upon failure, it still attempts further importing before loading - I haven't looked into that too far.

Attachments (1)

18730.patch (1.4 KB) - added by kurtpayne 4 years ago.
Determine file extension based on content type header

Download all attachments as: .zip

Change History (7)

#1 @JustinSainton
4 years ago

  • Cc JustinSainton added

Probably not relevant for your use case, but I was able to get around a similar bug by modifying the wp_check_filetype_and_ext filter as such -

add_filter( 'wp_check_filetype_and_ext', 'gd_pdf_bypass' );

function gd_pdf_bypass( $filearray ) {
    $filearray['type'] = 'image/jpeg';
    $filearray['ext'] = 1;
    return $filearray;

Granted, still doesn't attach properly (not necessary in my use case), but I'm able to get it to return the proper <img> tag and upload to the server with the override.

Version 0, edited 4 years ago by JustinSainton (next)

4 years ago

Determine file extension based on content type header

#2 @kurtpayne
4 years ago

  • Cc kpayne@… added
  • Keywords has-patch added
  • Version set to 2.9

#3 @c3mdigital
3 years ago

  • Cc chris@… added

I'm having this issue with some of my users in my media-tools plugin which imports and attaches external images. I'm going to work on refreshing the patch for 3.5.

#4 @nacin
22 months ago

  • Component changed from Warnings/Notices to Media

#5 @wonderboymusic
17 months ago

  • Milestone changed from Awaiting Review to Future Release

#6 @chriscct7
3 weeks ago

  • Keywords needs-refresh added
Note: See TracTickets for help on using tickets.