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

Escape '>' in XmlSerializer #177

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

terencehonles
Copy link

fixes: #165

Not sure if this repository is active anymore since it's not been touched in awhile and there seems to be a number of open PRs, but @sbaramov it's probably easiest to create a PR in the future 🙂 (I attributed you as the commit author).

I noticed the same thing and figured it must just be valid XML 🤷 but was looking at open issues and PR and saw #165

@terencehonles
Copy link
Author

It looks like this is not an oversight (and what I figured might be the case), but it looks like ">" "may" be escaped https://www.w3.org/TR/xml/#NT-AttValue

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

Successfully merging this pull request may close these issues.

'>' are not escaped by the XmlSerializer
2 participants