Thread overview
[Issue 17587] JSONOptions: No forward slash encoding
Jul 03, 2017
Vladimir Panteleev
Jul 04, 2017
Andre
Jul 05, 2017
Andre
Jul 05, 2017
ag0aep6g@gmail.com
Jul 05, 2017
Adam D. Ruppe
July 03, 2017
https://issues.dlang.org/show_bug.cgi?id=17587

Vladimir Panteleev <dlang-bugzilla@thecybershadow.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dlang-bugzilla@thecybershad
                   |                            |ow.net

--- Comment #1 from Vladimir Panteleev <dlang-bugzilla@thecybershadow.net> ---
Is not escaping / desired for purely a size optimization, or is the escaping actually breaking something?

--
July 04, 2017
https://issues.dlang.org/show_bug.cgi?id=17587

--- Comment #2 from Andre <andre@s-e-a-p.de> ---
In my AWS SDK I had the issue that \/ was not accepted while passing the JSON
as console argument to the AWS console client. In the meantime I save the JSON
as file and pass the file name to the AWS console client. Here the \/ is
accepted quite fine.
I tried to reactive the old behavior but wasn't able, because passing JSON as
console argument is quite another beast...

My other use cases is purely optical. In an http server I generate JSON
documents on the fly which are outputed to the browser. One example is an
ODATA2 server framework.
Something similiar to this one
http://services.odata.org/V2/Northwind/Northwind.svc/Customers?$FORMAT=JSON

With the right Chrome plugin, the links are clickable but I have to replace \/ in http://...

--
July 05, 2017
https://issues.dlang.org/show_bug.cgi?id=17587

--- Comment #3 from Andre <andre@s-e-a-p.de> ---
I just noticed another scenario in which I face the issue. I have a d application which adapt a dub.json configuration file. It adds some some sub packages which are created via my application.

I read the file as text, parses the text into JSON (std.json) adapt the JSON
and store it again as file.
If the file contains a homepage attribute, it will be rewritten as
http:\/\/...\/...  This rewriting is not intended.

--
July 05, 2017
https://issues.dlang.org/show_bug.cgi?id=17587

ag0aep6g@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ag0aep6g@gmail.com

--- Comment #4 from ag0aep6g@gmail.com ---
Pull request to move this forward to some resolution: https://github.com/dlang/phobos/pull/5552

--
July 05, 2017
https://issues.dlang.org/show_bug.cgi?id=17587

Adam D. Ruppe <destructionator@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |destructionator@gmail.com

--- Comment #5 from Adam D. Ruppe <destructionator@gmail.com> ---
> In my AWS SDK I had the issue that \/ was not accepted while passing the JSON as console argument to the AWS console client.

Sounds like you should be shell encoding the data first anyway!

--
July 21, 2017
https://issues.dlang.org/show_bug.cgi?id=17587

--- Comment #6 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/0503b9b4a55512335ce038a34aa57a137b76753d add JSONOptions.doNotEscapeSlashes

Fixes issue 17587 (enhancement request).

https://github.com/dlang/phobos/commit/987ebc54baffe1d8cf41a560ab7a746721e1c7de Merge pull request #5552 from aG0aep6G/17587

don't escape slash in toJSON (issue 17587)

--
July 21, 2017
https://issues.dlang.org/show_bug.cgi?id=17587

github-bugzilla@puremagic.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--
August 16, 2017
https://issues.dlang.org/show_bug.cgi?id=17587

--- Comment #7 from github-bugzilla@puremagic.com ---
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/0503b9b4a55512335ce038a34aa57a137b76753d add JSONOptions.doNotEscapeSlashes

https://github.com/dlang/phobos/commit/987ebc54baffe1d8cf41a560ab7a746721e1c7de Merge pull request #5552 from aG0aep6G/17587

--
January 05, 2018
https://issues.dlang.org/show_bug.cgi?id=17587

--- Comment #8 from github-bugzilla@puremagic.com ---
Commits pushed to dmd-cxx at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/0503b9b4a55512335ce038a34aa57a137b76753d add JSONOptions.doNotEscapeSlashes

https://github.com/dlang/phobos/commit/987ebc54baffe1d8cf41a560ab7a746721e1c7de Merge pull request #5552 from aG0aep6G/17587

--