Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handling of \r\n in MediaTrace XML output #550

Open
JeromeMartinez opened this issue Feb 27, 2017 · 5 comments
Open

Handling of \r\n in MediaTrace XML output #550

JeromeMartinez opened this issue Feb 27, 2017 · 5 comments
Assignees

Comments

@JeromeMartinez
Copy link
Member

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

@tribouille
Copy link
Contributor

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

@ablwr
Copy link
Member

ablwr commented Feb 27, 2017

👀

@JeromeMartinez
Copy link
Member Author

@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.

@dericed
Copy link
Contributor

dericed commented Jul 19, 2017

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 but contains MicroMediaTrace content.

@JeromeMartinez
Copy link
Member Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants