Merge pull request #38 from ConductorOne/bt/check_null

btipling · web-flow · commit eaf18fa5814f · 2024-12-04T16:58:37.000-08:00
[BB-195] check for the "null" string
diff --git a/pkg/snowflake/client.go b/pkg/snowflake/client.go
@@ -114,7 +114,7 @@ func (m *ResultSetMetadata) GetBoolValueFromRow(row []string, key string) (bool,
 	}
 
 	// "NULL"-ish case
-	if row[i] == "" {
+	if row[i] == "" || row[i] == "null" {
 		return false, nil
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,7 @@ func (m *ResultSetMetadata) GetBoolValueFromRow(row []string, key string) (bool,`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`// "NULL"-ish case`
`117`		`- if row[i] == "" {`
	`117`	`+ if row[i] == "" \|\| row[i] == "null" {`
`118`	`118`	`return false, nil`
`119`	`119`	`}`
`120`	`120`