hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-04-22 23:35:14 +02:00
Code Issues Packages Projects Releases Wiki Activity

avoid contractions

Browse Source
This commit is contained in:
erik-krogh
2023-05-17 08:42:45 +02:00
parent 83ca1495e0
commit 480e71fd69
4 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
javascript/ql/src/Performance/PolynomialReDoS.qhelp
View File

@@ -103,7 +103,7 @@ text.replace(/^\s+|\s+$/g, ''); // BAD</sample>
<example>
<p>
Sometimes it's unclear how a regular expression can be rewritten to
Sometimes it is unclear how a regular expression can be rewritten to
avoid the problem. In such cases, it often suffices to limit the
length of the input string. For instance, the following complicated
regular expression is used to match numbers, and on some non-number
@@ -114,7 +114,7 @@ text.replace(/^\s+|\s+$/g, ''); // BAD</sample>
/^(\+|-)?(\d+|(\d*\.\d*))?(E|e)?([-+])?(\d+)?$/.test(str) // BAD</sample>
<p>
It's not immediately obvious how to rewrite this regular expression
It is not immediately obvious how to rewrite this regular expression
to avoid the problem. However, it might be fine to limit the length
to 1000 characters, which will always finish in a reasonable amount
of time.