Home Original page

util: escape symbol and non-enumerable keys by BridgeAR · Pull Request #22300 · nodejs/node

@nodejs-github-bot added the util

Issues and PRs related to the built-in util module.

label

Aug 13, 2018

@BridgeAR 

These keys require escaping as they might also contain line breaks
and other special characters.

@BridgeAR BridgeAR added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Aug 17, 2018

benjamingr

mcollina

BridgeAR added a commit to BridgeAR/node that referenced this pull request

Aug 19, 2018 
These keys require escaping as they might also contain line breaks
and other special characters.

PR-URL: nodejs#22300
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

targos pushed a commit that referenced this pull request

Aug 19, 2018 
These keys require escaping as they might also contain line breaks
and other special characters.

PR-URL: #22300
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

targos pushed a commit that referenced this pull request

Sep 3, 2018 
These keys require escaping as they might also contain line breaks
and other special characters.

PR-URL: #22300
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

This was referenced

Sep 6, 2018