upnpserver icon indicating copy to clipboard operation
upnpserver copied to clipboard

Filter xml attributes

Open s-leger opened this issue 9 years ago • 4 comments

Upnp LastChange event model make a wide use of "val" complex attribute including embedding xml. Xmltojs currently doesn't filter attributes values so we need to escape those manually.

Pushed a pull request on xmltojs. Now wait and see.

s-leger avatar Oct 14 '15 14:10 s-leger

Is-it the same pull request as you ?

https://github.com/davidcalhoun/jstoxml/pull/9

Good day

oeuillot avatar Oct 14 '15 15:10 oeuillot

Hi, fix the same problem, but in a more complicated fashion (separated option for attributes).

davidcalhoun/jstoxml#20

s-leger avatar Oct 14 '15 15:10 s-leger

Yes It is more complicated, but it is older :-p

It seems that I have the same problem in a very huge list of movies.
I don't find where the XML is invalid, because there is only one Android client which detects this problem. The attributes encoding can be the explanation.

So, we can fork it (or copy jstoxml.js into our project), and when the pull request is accepted, we will reference the original implementation in the package.json !?

Olivier

oeuillot avatar Oct 14 '15 15:10 oeuillot

Go for a patched copy, until either solution was merged. Failure with parsing may also be caused by some special chars in regular text. Got this with a 1500+ movie list from ps3 media server submitted as playlist :) . With such huge list you may also facing with memory limitations issues.

s-leger avatar Oct 14 '15 15:10 s-leger