MediaConch_SourceCode icon indicating copy to clipboard operation
MediaConch_SourceCode copied to clipboard

Handling of \r\n in MediaTrace XML output

Open JeromeMartinez opened this issue 8 years ago • 5 comments

MediaInfoLib was buggy with \r or \n, being fixed, but after that MediaConch does not show the corresponding MediaTrace (and downloaded file is also not the right one) Test file with several forbidden XML characters

JeromeMartinez avatar Feb 27 '17 10:02 JeromeMartinez

Problem is due to the XSL which transforms MMT to MT. Can be seen using mediaconch -fx -mmt file and mediaconch -fx -mt file

tribouille avatar Feb 27 '17 14:02 tribouille

👀

ablwr avatar Feb 27 '17 14:02 ablwr

@dericed here is the output with latest modifications, I think \r \n are correctly handled, as well as other XML specials chars (<, >...): test_XML.mmt.xml.zip Please update the XSL.

JeromeMartinez avatar Feb 27 '17 15:02 JeromeMartinez

Is still an issue? I'm having trouble to recreate with 17.05.20170609. Also the attached test_XML.mmt.xml is incorrect because it uses <MediaTrace> but contains MicroMediaTrace content.

dericed avatar Jul 19 '17 05:07 dericed

MediaInfo library filters such chars as a workaround. Is your XSL ok when you check with an hand crafted XML containing such chars?

JeromeMartinez avatar Jul 19 '17 09:07 JeromeMartinez