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

Request: XML Parser should support CDATA #26

Closed
dcornejo opened this issue Jun 7, 2018 · 1 comment
Closed

Request: XML Parser should support CDATA #26

dcornejo opened this issue Jun 7, 2018 · 1 comment

Comments

@dcornejo
Copy link
Contributor

dcornejo commented Jun 7, 2018

There were some changes recently to protect data being passed through clixon from being misinterpreted as XML and breaking the parser (thanks for that).

We are still having problems because you need to keep track of whether a string has been encoded or decoded already - for example if I start with the string "%" and using URI encoding the encoded result is "%25" if that gets encoded again, I get "%2525" and when I'm decoding I have no way to tell whether the original was "%25" or "%"... (the same thing can happen with XML encoding too)

We believe that the best way to deal with this is to use CDATA - which essentially "quotes" the data and tells the parser to skip on to the end marker. There's no laborious or inefficient en/decoding and we can pass it through as many time as we like and it's not altered.

So, we're requesting you implement CDATA in clixon,

thanks

@olofhagsand
Copy link
Member

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

2 participants