diff options
| author | toasted-nutbread <toasted-nutbread@users.noreply.github.com> | 2023-12-19 00:33:38 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-12-19 05:33:38 +0000 |
| commit | 1ced9aafc00c10992bab8bd3f1b6b1397f05b7b9 (patch) | |
| tree | 305bb2b3bfc7fc3b051ee1cd3d1c35f442af0de4 /ext/js/pages/settings/backup-controller.js | |
| parent | 5f96276fda93dcad39f2165fd3c8d890aa5f9be5 (diff) | |
Make JSON.parse usage safer (#373)
* Make JSON.parse usage safer
* Fix any type
* Add readResponseJson
* Use readResponseJson
* Additional updates
* Rename files
* Add types
Diffstat (limited to 'ext/js/pages/settings/backup-controller.js')
| -rw-r--r-- | ext/js/pages/settings/backup-controller.js | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ext/js/pages/settings/backup-controller.js b/ext/js/pages/settings/backup-controller.js index 85803077..c539bdfe 100644 --- a/ext/js/pages/settings/backup-controller.js +++ b/ext/js/pages/settings/backup-controller.js @@ -18,6 +18,7 @@ import {Dexie} from '../../../lib/dexie.js'; import {isObject, log} from '../../core.js'; +import {parseJson} from '../../core/json.js'; import {OptionsUtil} from '../../data/options-util.js'; import {ArrayBufferUtil} from '../../data/sandbox/array-buffer-util.js'; import {querySelectorNotNull} from '../../dom/query-selector.js'; @@ -427,7 +428,8 @@ export class BackupController { if (this._optionsUtil === null) { throw new Error('OptionsUtil invalid'); } const dataString = ArrayBufferUtil.arrayBufferUtf8Decode(await this._readFileArrayBuffer(file)); - const data = JSON.parse(dataString); + /** @type {import('backup-controller').BackupData} */ + const data = parseJson(dataString); // Type check if (!isObject(data)) { |