NEOS-1634: Fixes AnonymizeSingle nil pointer exception on error (#2975)

nucleuscloud · Nov 21, 2024 · de249f9 · de249f9
1 parent a6e80c7
commit de249f9
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/backend/dev/build/Dockerfile.dev b/backend/dev/build/Dockerfile.dev
@@ -8,7 +8,7 @@ RUN apt-get update \
   apt-get install --no-install-recommends -y \
     build-essential
 
-RUN go install github.com/air-verse/air@latest
+RUN go install github.com/air-verse/air@v1.52.3
 
 # Copy the Go Modules manifests
 COPY go.mod go.sum ./

diff --git a/backend/internal/cmds/mgmt/serve/connect/cmd.go b/backend/internal/cmds/mgmt/serve/connect/cmd.go
@@ -114,6 +114,10 @@ func serve(ctx context.Context) error {
 	}
 	slogger.Debug(fmt.Sprintf("ee license enabled: %t", eelicense.IsValid()))
 
+	if getIsNeosyncCloud() {
+		slogger.Debug("neosync cloud is enabled")
+	}
+
 	mux := http.NewServeMux()
 
 	services := []string{

diff --git a/backend/services/mgmt/v1alpha1/anonymization-service/anonymization.go b/backend/services/mgmt/v1alpha1/anonymization-service/anonymization.go
@@ -192,7 +192,9 @@ func (s *Service) AnonymizeSingle(
 
 	outputData, err := anonymizer.AnonymizeJSONObject(req.Msg.InputData)
 	if err != nil {
-		outputErrorCounter.Add(ctx, int64(1), metric.WithAttributes(labels...))
+		if outputErrorCounter != nil {
+			outputErrorCounter.Add(ctx, int64(1), metric.WithAttributes(labels...))
+		}
 		return nil, err
 	}
 

diff --git a/worker/dev/build/Dockerfile.dev b/worker/dev/build/Dockerfile.dev
@@ -8,7 +8,7 @@ RUN apt-get update \
   apt-get install --no-install-recommends -y \
     build-essential
 
-RUN go install github.com/air-verse/air@latest
+RUN go install github.com/air-verse/air@@v1.52.3
 
 # Copy the Go Modules manifests
 COPY go.mod go.sum ./