Make WordPress Core

Opened 6 years ago

Closed 5 years ago

Last modified 4 years ago

#43836 closed defect (bug) (fixed)

Update or patch getID3 library

Reported by: sanchothefat's profile sanchothefat Owned by: joemcgill's profile joemcgill
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Media Keywords: needs-testing has-patch
Focuses: Cc:

Description

The getID3 library is a little out of date now and currently has the following defect.

File names are sanitised in such a way that a double slash (or directory separator) eg. // is replaced by a single instance of that character.

For file stream wrappers this is broken because a path like s3://example is changed to s3:/example.

The latest version of the library doesn't have this issue (the offending line is commented out) but requires PHP 5.3.

We should patch the offending line or update the entire library.

Attachments (2)

43836.patch (591 bytes) - added by sanchothefat 6 years ago.
Basic patch, removes offending line
43836.diff (371.3 KB) - added by desrosj 5 years ago.

Download all attachments as: .zip

Change History (16)

@sanchothefat
6 years ago

Basic patch, removes offending line

#1 @sanchothefat
6 years ago

  • Keywords has-patch added; needs-patch removed

#2 @swissspidy
6 years ago

#44819 was marked as a duplicate.

#3 @pento
6 years ago

  • Version trunk deleted

#4 @joemcgill
6 years ago

  • Milestone changed from Awaiting Review to 5.3
  • Owner set to joemcgill
  • Status changed from new to assigned

The latest version of the getID3 library as of today is 1.9.17. The version shipped in core is 1.9.14 since [41196]. Let's update the whole library to the latest version rather than patching specific issues.

#5 @desrosj
5 years ago

  • Keywords needs-patch added; has-patch removed

#6 @desrosj
5 years ago

  • Keywords needs-refresh added; needs-patch removed

@desrosj
5 years ago

#7 @desrosj
5 years ago

  • Keywords needs-testing has-patch added; needs-refresh removed

43836.diff updates the entire library to 1.9.17. I have a test branch on my fork passing Travis.

There are some additional classes that are not included in WordPress. I'm assuming they are not included because core does not utilize them. Maybe someone with a little historical knowledge can weigh in.

#9 @jrf
5 years ago

Updating to 1.9.17 is not a good idea as there are PHP 7.4 related issues in the code which have not yet been addressed in 1.9.17.

The current master has been fixed for those issues and I've opened an issue asking the maintainer to tag a new release: https://github.com/JamesHeinrich/getID3/issues/200

#10 @desrosj
5 years ago

  • Milestone 5.3 deleted
  • Resolution set to duplicate
  • Status changed from assigned to closed

While not directly a duplicate, #47751 (more specifically [46112]) has updated the getID3 library to the latest version, which is now 1.9.18 and includes PHP 7.4 compatibility issues.

#11 @jorbin
5 years ago

  • Resolution changed from duplicate to fixed

In 46166:

External Library: Update getid3 to 1.9.18

Follow up to r46112 as getid3 has tagged an official release.

Changes: https://github.com/JamesHeinrich/getID3/compare/00f3fbfd77e583099ca70a3cf0bc092e113d2b20...v1.9.18

Props desrosj, jkitchen.
Fixes #47751 #40883 #43836.

#12 @stokim
5 years ago

Hi @jorbin
getid3 was updated on Dec 2019 to 1.9.19, and this is the latest version with many bug fixes.
v1.9.19
@JamesHeinrich JamesHeinrich released this on Dec 18, 2019 · 49 commits to master since this release

1.9.19: [2019-12-17] James Heinrich :: 1.9.19-201912131005

add placeholder support for WTV (Windows Recorded TV Show)
bugfix (G:210) PHP 7.4 deprecated get_magic_quotes
bugfix (G:207) improved LAME version string parsing
bugfix (G:206) inverted logic in CopyTagsToComments
bugfix (G:203) use getimagesizefromstring if available
Quicktime decode 'uuid' atom for 360fly cameras
=========================================================================================
I hope it will be reflected in the next update.
Many thanks :)

#13 @SergeyBiryukov
4 years ago

#40883 was marked as a duplicate.

#14 @SergeyBiryukov
4 years ago

  • Milestone set to 5.3
Note: See TracTickets for help on using tickets.