diff options
author | Antoine du Hamel <duhamelantoine1995@gmail.com> | 2022-06-16 01:21:57 +0300 |
---|---|---|
committer | Danielle Adams <adamzdanielle@gmail.com> | 2022-06-16 04:28:47 +0300 |
commit | ac9599a7182505a78aaf6ab031b62f4408f7cdcd (patch) | |
tree | 224f68adc60da00924f58a4202526301fac620b3 /src/uv.cc | |
parent | 0bb84b09a55eb1605b8e231529d10f15aa9ed7f1 (diff) |
tools: report unsafe string and regex primordials as lint errors
| The string method | looks up the property |
| ----------------------------- | --------------------- |
| `String.prototype.match` | `Symbol.match` |
| `String.prototype.matchAll` | `Symbol.matchAll` |
| `String.prototype.replace` | `Symbol.replace` |
| `String.prototype.replaceAll` | `Symbol.replace` |
| `String.prototype.search` | `Symbol.search` |
| `String.prototype.split` | `Symbol.split` |
Functions that lookup the `exec` property on the prototype chain:
* `RegExp.prototype[Symbol.match]`
* `RegExp.prototype[Symbol.matchAll]`
* `RegExp.prototype[Symbol.replace]`
* `RegExp.prototype[Symbol.search]`
* `RegExp.prototype[Symbol.split]`
* `RegExp.prototype.test`
`RegExp.prototype[Symbol.replace]` and `RegExp.prototype[Symbol.split]`
are still allowed for a lack of a better solution.
PR-URL: https://github.com/nodejs/node/pull/43393
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'src/uv.cc')
0 files changed, 0 insertions, 0 deletions