Merge pull request #55 from statsig-io/fix-eq-neq-operator

fix eq and neq operators

evaluator.go

@@ -279,10 +279,19 @@ func (e *evaluator) evalCondition(user User, cond configCondition) *evalResult {
 		pass = matched
 
 	// strict equality
-	case "eq":
-		pass = reflect.DeepEqual(value, cond.TargetValue)
-	case "neq":
-		pass = !reflect.DeepEqual(value, cond.TargetValue)
+	case "eq", "neq":
+		equal := false
+		// because certain user values are of string type, which cannot be nil, we should check for both nil and empty string
+		if cond.TargetValue == nil {
+			equal = value == nil || value == ""
+		} else {
+			equal = reflect.DeepEqual(value, cond.TargetValue)
+		}
+		if op == "eq" {
+			pass = equal
+		} else {
+			pass = !equal
+		}
 
 	// time
 	case "before":

transport.go

@@ -34,7 +34,7 @@ func newTransport(secret string, options *Options) *transport {
 
 	return &transport{
 		api:      api,
-		metadata: statsigMetadata{SDKType: "go-sdk", SDKVersion: "1.2.0"},
+		metadata: statsigMetadata{SDKType: "go-sdk", SDKVersion: "1.2.1"},
 		sdkKey:   secret,
 		client:   &http.Client{},
 		options:  options,