Skip to content
New issue

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

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

Already on GitHub? # to your account

Invalid XML characters in JSON input #1667

Closed
michaelhkay opened this issue Dec 20, 2024 · 1 comment · Fixed by #1669
Closed

Invalid XML characters in JSON input #1667

michaelhkay opened this issue Dec 20, 2024 · 1 comment · Fixed by #1669
Labels
Editorial Minor typos, wording clarifications, example fixes, etc. PR Pending A PR has been raised to resolve this issue XQFO An issue related to Functions and Operators

Comments

@michaelhkay
Copy link
Contributor

We have changed the data model (§2.8.4) (see PR #546) so that implementations may allow characters that are not valid XML characters.

We have not explored the impact of this change on parse-json(), which is one of the obvious places where non-XML characters may arise. For example, JSON allows unescaped C1 control characters.

(Note however that the data model explicitly bans unpaired surrogates, and I think that rule should apply to parse-json() even though the JSON grammar allows it).

Hopefully it only requires clarification notes to be added to the spec, and not any substantive change.

@ChristianGruen
Copy link
Contributor

Good point. I have observed other cases in which a clarification would be helpful (I should spend some time to look at them closer).

@ChristianGruen ChristianGruen added XQFO An issue related to Functions and Operators Editorial Minor typos, wording clarifications, example fixes, etc. labels Dec 20, 2024
@michaelhkay michaelhkay added the PR Pending A PR has been raised to resolve this issue label Dec 31, 2024
@ndw ndw closed this as completed in #1669 Jan 7, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Editorial Minor typos, wording clarifications, example fixes, etc. PR Pending A PR has been raised to resolve this issue XQFO An issue related to Functions and Operators
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants