Merge pull request #4 from Galactica-corp/fix-issue2

Remove personal info from logs
SwissBorg · Sep 24, 2024 · a5b62df · a5b62df
2 parents 8517ca4 + 8936e7c
commit a5b62df
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/internal/api/handlers.go b/internal/api/handlers.go
@@ -41,7 +41,10 @@ func (h *Handlers) GenerateCert(c echo.Context) error {
 		})
 	}
 
-	log.Printf("REQUEST: %+v", req)
+	log.
+		WithField("holderCommitment", req.HolderCommitment).
+		WithField("userID", req.UserID).
+		Info("request")
 
 	if err := c.Validate(req); err != nil {
 		log.WithError(err).Error("validate gen cert request")
@@ -178,7 +181,9 @@ func (h *Handlers) GetCert(c echo.Context) error {
 		})
 	}
 
-	log.Printf("REQUEST: %+v", req)
+	log.
+		WithField("userID", req.UserID).
+		Info("request")
 
 	certificate, err := readCertFromDB(h.inMem, req.UserID)
 	if err != nil {