Address comments

Napalys · Napalys · commit 68a9dd9f9e72 · 2025-05-01T11:19:41.000+02:00
diff --git a/javascript/ql/lib/semmle/javascript/frameworks/Fastify.qll b/javascript/ql/lib/semmle/javascript/frameworks/Fastify.qll
@@ -169,9 +169,7 @@ module Fastify {
 
     override string getRelativePath() { result = this.getArgument(0).getStringValue() }
 
-    override Http::RequestMethodName getHttpMethod() {
-      if this.getMethodName() = "all" then any() else result = this.getMethodName().toUpperCase()
-    }
+    override Http::RequestMethodName getHttpMethod() { result = this.getMethodName().toUpperCase() }
   }
 
   private class AddHookRouteSetup extends Routing::RouteSetup::MethodCall instanceof RouteSetup {

Original file line number	Diff line number	Diff line change
`@@ -169,9 +169,7 @@ module Fastify {`
`169`	`169`
`170`	`170`	`override string getRelativePath() { result = this.getArgument(0).getStringValue() }`
`171`	`171`
`172`		`- override Http::RequestMethodName getHttpMethod() {`
`173`		`- if this.getMethodName() = "all" then any() else result = this.getMethodName().toUpperCase()`
`174`		`- }`
	`172`	`+ override Http::RequestMethodName getHttpMethod() { result = this.getMethodName().toUpperCase() }`
`175`	`173`	`}`
`176`	`174`
`177`	`175`	`private class AddHookRouteSetup extends Routing::RouteSetup::MethodCall instanceof RouteSetup {`