Fixes bug with filter-alerts install panics on null throttleTimeSeconds (#190)

jswoods · web-flow · commit 1b8929d7ae9c · 2025-06-02T08:44:10.000+02:00
diff --git a/internal/api/filter-alerts.go b/internal/api/filter-alerts.go
@@ -87,7 +87,7 @@ func (fa *FilterAlerts) Create(searchDomainName string, newFilterAlert *FilterAl
 		newFilterAlert.Enabled,
 		ownershipRunAsID,
 		newFilterAlert.ThrottleField,
-		*newFilterAlert.ThrottleTimeSeconds,
+		newFilterAlert.ThrottleTimeSeconds,
 		humiographql.QueryOwnershipType(newFilterAlert.QueryOwnershipType),
 	)
 	if err != nil {
diff --git a/internal/api/humiographql/graphql/filter-alerts.graphql b/internal/api/humiographql/graphql/filter-alerts.graphql
@@ -39,7 +39,7 @@ mutation CreateFilterAlert(
     $Enabled: Boolean!
     $RunAsUserID: String
     $ThrottleField: String
-    $ThrottleTimeSeconds: Long!
+    $ThrottleTimeSeconds: Long
     $QueryOwnershipType: QueryOwnershipType!
 ) {
     createFilterAlert(input: {
@@ -80,4 +80,4 @@ query GetFilterAlertByID(
             ...FilterAlertDetails
         }
     }
-}
+}
diff --git a/internal/api/humiographql/humiographql.go b/internal/api/humiographql/humiographql.go

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ func (fa FilterAlerts) Create(searchDomainName string, newFilterAlert FilterAl`
`87`	`87`	`newFilterAlert.Enabled,`
`88`	`88`	`ownershipRunAsID,`
`89`	`89`	`newFilterAlert.ThrottleField,`
`90`		`- *newFilterAlert.ThrottleTimeSeconds,`
	`90`	`+ newFilterAlert.ThrottleTimeSeconds,`
`91`	`91`	`humiographql.QueryOwnershipType(newFilterAlert.QueryOwnershipType),`
`92`	`92`	`)`
`93`	`93`	`if err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ mutation CreateFilterAlert(`
`39`	`39`	`$Enabled: Boolean!`
`40`	`40`	`$RunAsUserID: String`
`41`	`41`	`$ThrottleField: String`
`42`		`- $ThrottleTimeSeconds: Long!`
	`42`	`+ $ThrottleTimeSeconds: Long`
`43`	`43`	`$QueryOwnershipType: QueryOwnershipType!`
`44`	`44`	`) {`
`45`	`45`	`createFilterAlert(input: {`
`@@ -80,4 +80,4 @@ query GetFilterAlertByID(`
`80`	`80`	`...FilterAlertDetails`
`81`	`81`	`}`
`82`	`82`	`}`
`83`		`-}`
	`83`	`+}`