Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#32082 closed defect (bug) (duplicate)

Not accounting for Params in src in wp_video_shortcode

Reported by: cartpauj Owned by:
Priority: normal Milestone:
Component: Shortcodes Version: 4.1.2
Severity: normal Keywords:
Cc: Focuses:

Description

Trying to pass a src with URI params fails the ext matching here: https://core.trac.wordpress.org/browser/tags/4.1.2/src/wp-includes/media.php#L1761

The issue is due to the regex in wp_check_filetype here: https://core.trac.wordpress.org/browser/tags/4.1.2/src/wp-includes/functions.php#L2052

It needs to be changed to something like:

$ext_preg = '!\.(' . $ext_preg . ‘)(\?.*)?$!i';

Example src URL that fails:
https://wordup.s3.amazonaws.com/99-right-curve-sequence.mp4?AWSAccessKeyId=KEY&Expires=1429813612&Signature=SIG

Attachments (1)

32082.patch (1.3 KB ) - added by ocean90 10 years ago.

Download all attachments as: .zip

Change History (2)

#1 @johnbillion
11 years ago

  • Milestone Awaiting Review
  • Resolutionduplicate
  • Status newclosed

Duplicate of #30377.

@ocean90
10 years ago

Note: See TracTickets for help on using tickets.