Merge pull request #1551 from Fdawgs/ref/regex

perf: remove unused regex capture groups
tediousjs · Sep 12, 2023 · c4544be · c4544be
2 parents 2e33052 + 9a240ac
commit c4544be
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/base/prepared-statement.js b/lib/base/prepared-statement.js
@@ -91,7 +91,7 @@ class PreparedStatement extends EventEmitter {
    */
 
   input (name, type) {
-    if ((/(--| |\/\*|\*\/|')/).test(name)) {
+    if (/--| |\/\*|\*\/|'/.test(name)) {
       throw new PreparedStatementError(`SQL injection warning for param '${name}'`, 'EINJECT')
     }
 
@@ -144,7 +144,7 @@ class PreparedStatement extends EventEmitter {
    */
 
   output (name, type) {
-    if (/(--| |\/\*|\*\/|')/.test(name)) {
+    if (/--| |\/\*|\*\/|'/.test(name)) {
       throw new PreparedStatementError(`SQL injection warning for param '${name}'`, 'EINJECT')
     }
 

diff --git a/lib/base/request.js b/lib/base/request.js
@@ -107,7 +107,7 @@ class Request extends EventEmitter {
    */
 
   input (name, type, value) {
-    if ((/(--| |\/\*|\*\/|')/).test(name)) {
+    if (/--| |\/\*|\*\/|'/.test(name)) {
       throw new RequestError(`SQL injection warning for param '${name}'`, 'EINJECT')
     }
 
@@ -170,7 +170,7 @@ class Request extends EventEmitter {
   output (name, type, value) {
     if (!type) { type = TYPES.NVarChar }
 
-    if ((/(--| |\/\*|\*\/|')/).test(name)) {
+    if (/--| |\/\*|\*\/|'/.test(name)) {
       throw new RequestError(`SQL injection warning for param '${name}'`, 'EINJECT')
     }