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

quoted solidus is not unquoted #36

Closed
gfldex opened this issue Feb 10, 2018 · 5 comments
Closed

quoted solidus is not unquoted #36

gfldex opened this issue Feb 10, 2018 · 5 comments

Comments

@gfldex
Copy link
Collaborator

gfldex commented Feb 10, 2018

perl6 -MJSON::Fast -e 'dd from-json(q<"http:\/\/perl6.org">)'
# "http:\\\\/\\\\/perl6.org"
# expected: http://perl6.org

Apperently that is a thing: https://stackoverflow.com/questions/19176024/how-to-escape-special-characters-in-building-a-json-string#answer-27516892

@gfldex
Copy link
Collaborator Author

gfldex commented Feb 11, 2018

And that thing seams to be used widely by php-folk. Example: http://www.radio-browser.info/webservice/json/stations

@zoffixznet
Copy link
Contributor

zoffixznet commented Feb 23, 2018

I'm having the same problem:

2018.02.24 zoffix@VirtualBox~$ perl6 -MJSON::Fast -e 'say from-json q|"\/"|'
\\/
2018.02.24 zoffix@VirtualBox~$ perl -MMojo::JSON=from_json -wlE 'say from_json q|"\/"|'
/

Downgrading to a commit before 7bb2158 fixes the problem:

zef uninstall JSON::Fast
zef install https://github.com/timo/json_fast/archive/f8b5561fdb5882263898aa1e6b5d962a50898bb1.zip

@timo
Copy link
Owner

timo commented Feb 23, 2018

i believe i fixed this is release 0.9.10; please try it on your real-life data, too

@zoffixznet
Copy link
Contributor

Fixed for me.

@gfldex
Copy link
Collaborator Author

gfldex commented Feb 24, 2018

Fixed for me, tested with fairly large dataset.

@gfldex gfldex closed this as completed Feb 24, 2018
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

3 participants