Remove unused flow label

jeongsoolee09 · jeongsoolee09 · commit 584e6bcdf057 · 2025-05-15T11:14:18.000-04:00
diff --git a/javascript/frameworks/ui5/src/UI5LogInjection/UI5LogsToHttp.ql b/javascript/frameworks/ui5/src/UI5LogInjection/UI5LogsToHttp.ql
@@ -27,9 +27,7 @@ class ClientRequestInjectionVector extends DataFlow::Node {
 }
 
 class UI5LogEntryFlowState extends DataFlow::FlowLabel {
-  UI5LogEntryFlowState() {
-    this = ["not-logged-not-accessed", "logged-not-accessed", "logged-and-accessed"]
-  }
+  UI5LogEntryFlowState() { this = ["not-logged-not-accessed", "logged-and-accessed"] }
 }
 
 class UI5LogEntryToHttp extends TaintTracking::Configuration {
diff --git a/javascript/frameworks/ui5/test/queries/UI5LogInjection/log-entry-flows-to-sinks/UI5LogsToHttp.expected b/javascript/frameworks/ui5/test/queries/UI5LogInjection/log-entry-flows-to-sinks/UI5LogsToHttp.expected
@@ -1,9 +1,9 @@
-WARNING: type 'Configuration' has been deprecated and may be removed in future (UI5LogsToHttp.ql:35,33-61)
+WARNING: type 'Configuration' has been deprecated and may be removed in future (UI5LogsToHttp.ql:33,33-61)
 WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:29,36-55)
-WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:38,52-71)
-WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:44,47-66)
-WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:45,5-24)
-WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:71,50-69)
+WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:36,52-71)
+WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:42,47-66)
+WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:43,5-24)
+WARNING: type 'FlowLabel' has been deprecated and may be removed in future (UI5LogsToHttp.ql:69,50-69)
 nodes
 | webapp/controller/app.controller.js:11:11:11:21 | input: null |
 | webapp/controller/app.controller.js:17:13:17:48 | input |

Original file line number	Diff line number	Diff line change
`@@ -27,9 +27,7 @@ class ClientRequestInjectionVector extends DataFlow::Node {`
`27`	`27`	`}`
`28`	`28`
`29`	`29`	`class UI5LogEntryFlowState extends DataFlow::FlowLabel {`
`30`		`- UI5LogEntryFlowState() {`
`31`		`- this = ["not-logged-not-accessed", "logged-not-accessed", "logged-and-accessed"]`
`32`		`- }`
	`30`	`+ UI5LogEntryFlowState() { this = ["not-logged-not-accessed", "logged-and-accessed"] }`
`33`	`31`	`}`
`34`	`32`
`35`	`33`	`class UI5LogEntryToHttp extends TaintTracking::Configuration {`