chore: Adds errcheck lint check

omec-project · Jan 4, 2024 · ff55837 · ff55837
1 parent d871d55
commit ff55837
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 4 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -220,7 +220,7 @@ linters:
   enable:
     - gofmt
     - govet
-    # - errcheck
+    - errcheck
     - staticcheck
     - unused
     # - gosimple

diff --git a/factory/config.go b/factory/config.go
@@ -131,7 +131,10 @@ func (c *Config) updateConfig(commChannel chan *protos.NetworkSliceResponse, dbU
 				}
 
 			}
-			c.addSmPolicyInfo(ns, dbUpdateChannel)
+			err := c.addSmPolicyInfo(ns, dbUpdateChannel)
+			if err != nil {
+				logger.GrpcLog.Errorf("Error in adding sm policy info to db %v", err)
+			}
 		}
 		if minConfig == false {
 			// first slice Created

diff --git a/producer/data_repository.go b/producer/data_repository.go
@@ -57,8 +57,14 @@ func HandleCreateAccessAndMobilityData(request *http_wrapper.Request) *http_wrap
 
 // seems something which we should move to mongolib
 func toBsonM(data interface{}) (ret bson.M) {
-	tmp, _ := json.Marshal(data)
-	json.Unmarshal(tmp, &ret)
+	tmp, err := json.Marshal(data)
+	if err != nil {
+		logger.CfgLog.Infoln("marshal fail ", err)
+	}
+	err = json.Unmarshal(tmp, &ret)
+	if err != nil {
+		logger.CfgLog.Infoln("unmarshal fail ", err)
+	}
 	return
 }