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)
Change History (2)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Duplicate of #30377.