1056616 - Unify the use of JSON.parse and catch errors.

Status: RESOLVED FIXED

Description

[Alexis Metaireau (:alexis)]

It looks like we have 12 instances (in 9 files) to JSON.parse() in this repo, ?q="JSON.parse"&type=Code

We'd probably want to wrap them all in try...catch blocks, or at least the ones not in the /test/ dir.

    /loop/encrypt.js:47
    /loop/fxa.js:27
    /loop/routes/calls.js:252
    /loop/storage/redis.js:141
    /loop/storage/redis.js:198
    /loop/websockets.js:358
    /test/functional_test.js:1178
    /test/middlewares_test.js:58
    /test/storage_test.js:236
    /test/storage_test.js:260
    /test/websockets_test.js:74
    /test/websockets_test.js:90

Comment 1

[Rémy Hubscher (:natim)]

I checked them and all of them are find:

 - For redis we are sure it is JSON 
 - For encrypt it is throwing an error handled later
 - For fxa, I have made a patch for it.
 - For calls it is a JSON.parse(JSON.stringify())
 - For tests we want the error to be thrown.
 - For websocket it is handled on the decode call.

Comment 2

[Rémy Hubscher (:natim)]

Created attachment 8479660 [details] [review]
Link to Github PR — #193