[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 --- Comment #8 from Marco maic...@yahoo.com --- (In reply to Gerrit Notification Bot from comment #6) Change 130966 merged by jenkins-bot: Less false positives for MEDIATYPE_VIDEO https://gerrit.wikimedia.org/r/130966 It was merged last month but did not yet hit commons? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 --- Comment #9 from Brad Jorsch bjor...@wikimedia.org --- (In reply to Marco from comment #8) It was merged last month but did not yet hit commons? It was merged on June 29, which means that it will be deployed to WMF wikis with 1.24wmf12. That version went out to Commons yesterday; see https://www.mediawiki.org/wiki/MediaWiki_1.24/Roadmap for the full schedule. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 --- Comment #10 from Bawolff (Brian Wolff) bawolff...@gmail.com --- (In reply to Brad Jorsch from comment #9) (In reply to Marco from comment #8) It was merged last month but did not yet hit commons? It was merged on June 29, which means that it will be deployed to WMF wikis with 1.24wmf12. That version went out to Commons yesterday; see https://www.mediawiki.org/wiki/MediaWiki_1.24/Roadmap for the full schedule. Also it should be noted, it will only apply to new files (or files that you ?action=purge) -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 --- Comment #4 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 130966 had a related patch set uploaded by Brian Wolff: Less false positives for MEDIATYPE_VIDEO https://gerrit.wikimedia.org/r/130966 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 --- Comment #5 from Bawolff (Brian Wolff) bawolff...@gmail.com --- Just as an aside, on newer ffmepeg2theora, this probably won't happen as the first 255 bytes will be taken up by ogg skeleton. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 --- Comment #6 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 130966 merged by jenkins-bot: Less false positives for MEDIATYPE_VIDEO https://gerrit.wikimedia.org/r/130966 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 Bawolff (Brian Wolff) bawolff...@gmail.com changed: What|Removed |Added Status|PATCH_TO_REVIEW |NEW --- Comment #7 from Bawolff (Brian Wolff) bawolff...@gmail.com --- Resetting to new - the above patch helps the situation, but isn't a proper fix. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 --- Comment #3 from Marco maic...@yahoo.com --- Moved to Change-Id Ib39ed06f895624b55d15a001cb0a2cd3129f4bb0 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 Andre Klapper aklap...@wikimedia.org changed: What|Removed |Added Status|NEW |PATCH_TO_REVIEW -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 Derk-Jan Hartman hartman.w...@gmail.com changed: What|Removed |Added CC||hartman.w...@gmail.com --- Comment #2 from Derk-Jan Hartman hartman.w...@gmail.com --- A patch for this is submitted in https://github.com/wikimedia/mediawiki-core/pull/23 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 Andre Klapper aklap...@wikimedia.org changed: What|Removed |Added Priority|Unprioritized |Normal Version|unspecified |1.23-git -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63584] False positives for MEDIATYPE_VIDEO due to looking for string theora in (audio) ogg files
https://bugzilla.wikimedia.org/show_bug.cgi?id=63584 Bawolff (Brian Wolff) bawolff...@gmail.com changed: What|Removed |Added CC||aarcos.w...@gmail.com, ||bawolff...@gmail.com, ||fflo...@wikimedia.org, ||gti...@wikimedia.org, ||mtrac...@member.fsf.org Component|API |File management Summary|API responds with |False positives for |mediatype: VIDEO for an |MEDIATYPE_VIDEO due to |audio file |looking for string theora ||in (audio) ogg files --- Comment #1 from Bawolff (Brian Wolff) bawolff...@gmail.com --- Wow is the code to determine media type ever hacky... We determine if the media type is VIDEO, by seeing if the word theora appears in the first 256 bytes of the file (MimeMagic::getMediaType). In this case, the file begins with: 4f 67 67 53 00 02 00 00 00 00 00 00 00 00 ea 37 |OggS...7| 0010 77 5a 00 00 00 00 32 03 54 97 01 1e 01 76 6f 72 |wZ2.Tvor| 0020 62 69 73 00 00 00 00 02 44 ac 00 00 00 00 00 00 |bis.D...| 0030 80 38 01 00 00 00 00 00 b8 01 4f 67 67 53 00 00 |.8OggS..| 0040 00 00 00 00 00 00 00 00 ea 37 77 5a 01 00 00 00 |.7wZ| 0050 c2 13 86 6c 0f 4b ff ff ff ff ff ff ff ff ff ff |...l.K..| 0060 ff ff ff a9 03 76 6f 72 62 69 73 1d 00 00 00 58 |.vorbisX| 0070 69 70 68 2e 4f 72 67 20 6c 69 62 56 6f 72 62 69 |iph.Org libVorbi| 0080 73 20 49 20 32 30 30 37 30 36 32 32 01 00 00 00 |s I 20070622| 0090 1a 00 00 00 45 4e 43 4f 44 45 52 3d 66 66 6d 70 |ENCODER=ffmp| 00a0 65 67 32 74 68 65 6f 72 61 2d 30 2e 32 33 01 05 |eg2theora-0.23..| 00b0 76 6f 72 62 69 73 21 42 43 56 01 00 00 01 00 18 |vorbis!BCV..| 00c0 63 54 29 46 99 52 d2 4a 89 19 73 94 31 46 99 62 |cT)F.R.J..s.1F.b| [...] So the vorbis comment that the encoder is ffmpeg2theora-0.23 triggers MediaWiki to think the file is a video as the word theora appears at the beginning of the file. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l