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

No escape character support for JsonPath #3565

Open
nan01ab opened this issue Nov 3, 2024 · 3 comments
Open

No escape character support for JsonPath #3565

nan01ab opened this issue Nov 3, 2024 · 3 comments
Labels
Discussion Initial issue state - proposed but not yet accepted

Comments

@nan01ab
Copy link
Contributor

nan01ab commented Nov 3, 2024

JPathToken doesn't seem to handle the escape-character/escape sequence.

For example, JPathToken cannot process $.store.book['o\'clock']

Some basic escape characters may need to be handle here. (https://www.rfc-editor.org/rfc/rfc9535#section-2.3.1.2)

Where in the software does this update applies to?

  • Neo.Json
@nan01ab nan01ab added the Discussion Initial issue state - proposed but not yet accepted label Nov 3, 2024
@cschuchardt88
Copy link
Member

we use UTF8Strict. Which is UTF8 noBOM i believe with no fallbacks. This is one reason why.

@nan01ab
Copy link
Contributor Author

nan01ab commented Nov 3, 2024

we use UTF8Strict. Which is UTF8 noBOM i believe with no fallbacks. This is one reason why.

' just ASCII

@cschuchardt88
Copy link
Member

I been wanting to use the native json libraries System.Test.Json, for this reason. However all our classes would need to change, by adding attributes to know which properties to expose. However I think the biggest part is the VM. Seeing how no one wanted the VM changed in the pass. But now we are changing the VM so would be good make this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Initial issue state - proposed but not yet accepted
Projects
None yet
Development

No branches or pull requests

2 participants