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

Rework JSON and JavaScript escaping #726

Closed
GoogleCodeExporter opened this issue Mar 25, 2015 · 3 comments
Closed

Rework JSON and JavaScript escaping #726

GoogleCodeExporter opened this issue Mar 25, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

The JSON and JavaScript string escaping needs to be reworked.
 * they need to be separate, JavaScript needs to escape U+2028 and U+2029 while JSON does not (this also solve the dependency issue)
 * an optimized version for Pharo similar to what we have for XML and UTF-8 would be nice
 * do we really have to encode '</' as <\/'?
  * if it's inside a <script> it should work fine without it
  * if it's inside anything other and <script> or <style> HTML escaping should kick in
  * if it's inside an attribute value HTML escaping should kick in

Original issue reported on code.google.com by philippe...@gmail.com on 2 May 2012 at 6:35

@GoogleCodeExporter
Copy link
Author

Name: Seaside-Tests-JSON-pmm.5
Author: pmm
Time: 6 May 2012, 3:14:01 pm
UUID: 8a6808b3-a6b1-4558-a2a4-7018534850e4
Ancestors: Seaside-Tests-JSON-lr.4

- Issue 726:    Rework JSON and JavaScript escaping
- add string escaping tests

Original comment by philippe...@gmail.com on 6 May 2012 at 1:14

  • Changed state: Started
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Name: Seaside-JSON-Core-pmm.10
Author: pmm
Time: 6 May 2012, 3:57:30 pm
UUID: 0a67b0d2-3a62-4202-b05c-4993e0bdb39a
Ancestors: Seaside-JSON-Core-pmm.9

- Issue 726:    Rework JSON and JavaScript escaping
- introduce JSON encoder (should be a bit faster due to #notNil trick)

Name: Seaside-Pharo-JSON-Core-pmm.1
Author: pmm
Time: 6 May 2012, 4:00:26 pm
UUID: c4d1d5c2-39c6-4c6c-9d60-8e685bcc69a1
Ancestors: 

- Issue 726:    Rework JSON and JavaScript escaping
- add optimized Pharo implementation

Original comment by philippe...@gmail.com on 6 May 2012 at 2:01

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Name: JQuery-Tests-Core-pmm.135
Author: pmm
Time: 7 May 2012, 7:52:03 pm
UUID: 1feeed17-45f4-4436-9c2a-e3a70e38c4ae
Ancestors: JQuery-Tests-Core-pmm.134

- Issue 726:    Rework JSON and JavaScript escaping
- http://code.google.com/p/seaside/issues/detail?id=726
- update JavaScript part

Name: Javascript-Core-pmm.95
Author: pmm
Time: 7 May 2012, 7:52:22 pm
UUID: e6b56830-233a-4a7b-a4d9-83938b53e05b
Ancestors: Javascript-Core-pmm.94

- Issue 726:    Rework JSON and JavaScript escaping
- http://code.google.com/p/seaside/issues/detail?id=726
- update JavaScript part

Name: Javascript-Tests-Core-pmm.68
Author: pmm
Time: 7 May 2012, 7:52:37 pm
UUID: 1a101a86-13b7-4c80-8b68-4b59d8821d1c
Ancestors: Javascript-Tests-Core-pmm.67

- Issue 726:    Rework JSON and JavaScript escaping
- http://code.google.com/p/seaside/issues/detail?id=726
- update JavaScript part

Name: Prototype-Tests-Core-pmm.16
Author: pmm
Time: 7 May 2012, 7:52:52 pm
UUID: 56a66f8a-2f8c-496c-9c5e-9817ced699fb
Ancestors: Prototype-Tests-Core-pmm.15

- Issue 726:    Rework JSON and JavaScript escaping
- http://code.google.com/p/seaside/issues/detail?id=726
- update JavaScript part

Original comment by philippe...@gmail.com on 7 May 2012 at 5:53

  • Changed state: Fixed
  • Added labels: Type-CleanUp
  • Removed labels: Type-Feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant