summaryrefslogtreecommitdiff
path: root/lib/internal/debugger/inspect_client.js
Commit message (Collapse)AuthorAgeFilesLines
* debugger: add trailing commas in source filesAntoine du Hamel2023-02-221-1/+1
| | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/46714 Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com> Reviewed-By: Debadree Chatterjee <debadree333@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: James M Snell <jasnell@gmail.com>
* lib: enforce use of trailing commas for functionsAntoine du Hamel2023-02-141-1/+1
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/46629 Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
* lib: refactor to use letgdccwxx2021-10-091-5/+3
| | | | | | | | | move variable into each for loop PR-URL: https://github.com/nodejs/node/pull/40364 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Zijian Liu <lxxyxzj@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
* debugger: remove undefined parameterRich Trott2021-08-011-2/+2
| | | | | | | | | The data parameter of unpackError() is typically undefined. PR-URL: https://github.com/nodejs/node/pull/39570 Refs: https://github.com/nodejs/node-inspect/issues/101 Reviewed-By: Jan Krems <jan.krems@gmail.com> Reviewed-By: Michaƫl Zasso <targos@protonmail.com>
* debugger: validate sec-websocket-accept response headerChris Opperwall2021-07-181-4/+22
| | | | | | | | | | | | | | This addresses a TODO to validate that the sec-websocket-accept header in the WebSocket handshake response is valid. To do this we need to append the WebSocket GUID to the original key sent in sec-websocket-key, sha1 hash it, and then compare the base64 encoding with the value sent in the sec-websocket-accept response header. If they don't match, an error is thrown. PR-URL: https://github.com/nodejs/node/pull/39357 Refs: https://github.com/nodejs/node-inspect/pull/93 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* debugger: rename internal library for clarityRich Trott2021-06-201-0/+339
When I moved these files from node-inspect to Node.js core, I put them in lib/internal/inspector. That was a mistake. They should be in lib/internal/debugger. PR-URL: https://github.com/nodejs/node/pull/39080 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>