#43836 closed defect (bug) (fixed)
Update or patch getID3 library
Reported by: | sanchothefat | Owned by: | 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)
Change History (16)
#4
@
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.
#6
@
5 years ago
- Keywords needs-refresh added; needs-patch removed
For reference, here is a list of commits since 1.9.14: https://github.com/JamesHeinrich/getID3/compare/v1.9.14...v1.9.17
Also, the changelog: https://github.com/JamesHeinrich/getID3/blob/master/changelog.txt.
#7
@
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
@
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
@
5 years ago
- Milestone 5.3 deleted
- Resolution set to duplicate
- Status changed from assigned to closed
#12
@
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 :)
Basic patch, removes offending line