Abhishek Kondur 2 vuotta sitten
vanhempi
commit
a986db6775
4 muutettua tiedostoa jossa 11 lisäystä ja 15 poistoa
  1. 4 3
      auth/auth.go
  2. 1 2
      controllers/ext_client.go
  3. 1 1
      controllers/network.go
  4. 5 9
      controllers/user.go

+ 4 - 3
auth/auth.go

@@ -10,6 +10,7 @@ import (
 	"time"
 
 	"golang.org/x/crypto/bcrypt"
+	"golang.org/x/exp/slog"
 	"golang.org/x/oauth2"
 
 	"github.com/gorilla/websocket"
@@ -240,7 +241,7 @@ func HandleHeadlessSSO(w http.ResponseWriter, r *http.Request) {
 func addUser(email string) error {
 	var hasSuperAdmin, err = logic.HasSuperAdmin()
 	if err != nil {
-		logger.Log(1, "error checking for existence of admin user during OAuth login for", email, "; user not added")
+		slog.Error("error checking for existence of admin user during OAuth login for", "email", email, "error", err)
 		return err
 	} // generate random password to adapt to current model
 	var newPass, fetchErr = fetchPassValue("")
@@ -253,9 +254,9 @@ func addUser(email string) error {
 	}
 	if !hasSuperAdmin { // must be first attempt, create a superadmin
 		if err = logic.CreateSuperAdmin(&newUser); err != nil {
-			logger.Log(1, "error creating super admin from user,", email, "; user not added")
+			slog.Error("error creating super admin from user", "email", email, "error", err)
 		} else {
-			logger.Log(1, "superadmin created from user,", email, "; was first user added")
+			slog.Info("superadmin created from user", "email", email)
 		}
 	} else { // otherwise add to db as admin..?
 		// TODO: add ability to add users with preemptive permissions

+ 1 - 2
controllers/ext_client.go

@@ -94,8 +94,7 @@ func getAllExtClients(w http.ResponseWriter, r *http.Request) {
 	networksSlice := []string{}
 	marshalErr := json.Unmarshal([]byte(headerNetworks), &networksSlice)
 	if marshalErr != nil {
-		logger.Log(0, "error unmarshalling networks: ",
-			marshalErr.Error())
+		slog.Error("error unmarshalling networks", "error", marshalErr.Error())
 		logic.ReturnErrorResponse(w, r, logic.FormatError(marshalErr, "internal"))
 		return
 	}

+ 1 - 1
controllers/network.go

@@ -47,7 +47,7 @@ func getNetworks(w http.ResponseWriter, r *http.Request) {
 
 	allnetworks, err := logic.GetNetworks()
 	if err != nil && !database.IsEmptyRecord(err) {
-		logger.Log(0, r.Header.Get("user"), "failed to fetch networks: ", err.Error())
+		slog.Error("failed to fetch networks", "error", err.Error())
 		logic.ReturnErrorResponse(w, r, logic.FormatError(err, "internal"))
 		return
 	}

+ 5 - 9
controllers/user.go

@@ -208,9 +208,7 @@ func createSuperAdmin(w http.ResponseWriter, r *http.Request) {
 
 	err := json.NewDecoder(r.Body).Decode(&u)
 	if err != nil {
-
-		logger.Log(0, u.UserName, "error decoding request body: ",
-			err.Error())
+		slog.Error("error decoding request body", "error", err.Error())
 		logic.ReturnErrorResponse(w, r, logic.FormatError(err, "badrequest"))
 		return
 	}
@@ -222,19 +220,17 @@ func createSuperAdmin(w http.ResponseWriter, r *http.Request) {
 
 	err = logic.CreateSuperAdmin(&u)
 	if err != nil {
-		logger.Log(0, u.UserName, "failed to create admin: ",
-			err.Error())
+		slog.Error("failed to create admin", "error", err.Error())
 		logic.ReturnErrorResponse(w, r, logic.FormatError(err, "badrequest"))
 		return
 	}
-
 	logger.Log(1, u.UserName, "was made a super admin")
 	json.NewEncoder(w).Encode(logic.ToReturnUser(u))
 }
 
 // swagger:route POST /api/users/adm/transfersuperadmin user transferSuperAdmin
 //
-// Transfers suoeradmin role to an admin user.
+// Transfers superadmin role to an admin user.
 //
 //			Schemes: https
 //
@@ -330,11 +326,11 @@ func createUser(w http.ResponseWriter, r *http.Request) {
 
 	err = logic.CreateUser(&user)
 	if err != nil {
-		logger.Log(0, user.UserName, "error creating new user: ", err.Error())
+		slog.Error("error creating new user: ", "user", user.UserName, "error", err.Error())
 		logic.ReturnErrorResponse(w, r, logic.FormatError(err, "badrequest"))
 		return
 	}
-	logger.Log(1, user.UserName, "was created")
+	slog.Info("user was created", "username", user.UserName)
 	json.NewEncoder(w).Encode(logic.ToReturnUser(user))
 }