B: fix issues preventing logins from happening and add logging

ui/controllers/auth.go

@@ -26,11 +26,13 @@ func SignUpHandler (c *gin.Context) {
 	reqBody.Password = c.Request.FormValue("password")
 
 	if reqBody.Email == "" {
+		log.Warn("Request contained no email")
 		renderTempl(c, templates.SignupPage("Please provide an email"))
 		return
 	}
 
 	if reqBody.Password == "" {
+		log.Warn("Request contained no password")
 		renderTempl(c, templates.SignupPage("Please provide a password"))
 		return
 	}
@@ -38,11 +40,13 @@ func SignUpHandler (c *gin.Context) {
 	//Verify username and password
 	user, err := mongo.FindUserByEmail(reqBody.Email)
 	if err != nil {
+		log.WithError(err).Errorf("Failed to lookup user: %s", reqBody.Email)
 		renderTempl(c, templates.SignupPage("Error occured. Please try again later"))
 		return
 	}
 
 	if user != nil {
+		log.Warnf("User: %s, already exists", reqBody.Email)
 		renderTempl(c, templates.SignupPage(fmt.Sprintf("user already exists for %s", reqBody.Email)))
 		return
 	}
@@ -51,6 +55,7 @@ func SignUpHandler (c *gin.Context) {
 
 	passHash, err := bcrypt.GenerateFromPassword([]byte(reqBody.Password), 10)
 	if err != nil {
+		log.WithError(err).Errorf("Passowrd hash failed for user: %s", reqBody.Email)
 		renderTempl(c, templates.SignupPage("Signup failed. Please try again later"))
 		return
 	}
@@ -60,6 +65,7 @@ func SignUpHandler (c *gin.Context) {
 
 	err = mongo.SaveModel(user)
 	if err != nil {
+		log.WithError(err).Errorf("Failed to write user to DB for user: %s", reqBody.Email)
 		renderTempl(c, templates.SignupPage("Signup failed. Please try again later"))
 		return
 	}
@@ -72,8 +78,9 @@ func SignUpHandler (c *gin.Context) {
 		},
 	)
 
-	jwtStr, err := token.SignedString(conf.JwtSecret)
+	jwtStr, err := token.SignedString([]byte(conf.JwtSecret))
 	if err != nil {
+		log.WithError(err).Errorf("Failed to encode jwt for user: %s", reqBody.Email)
 		renderTempl(c, templates.SignupPage("Signup failed. Please try again later"))
 		return
 	}
@@ -94,11 +101,13 @@ func LoginHandler(c *gin.Context) {
 	reqBody.Password = c.Request.FormValue("password")
 
 	if reqBody.Email == "" {
+		log.Warn("Request contained no email")
 		renderTempl(c, templates.LoginPage("Please provide an email"))
 		return
 	}
 
 	if reqBody.Password == "" {
+		log.Warn("Request contained no password")
 		renderTempl(c, templates.LoginPage("Please provide a password"))
 		return
 	}
@@ -106,17 +115,20 @@ func LoginHandler(c *gin.Context) {
 	//Verify username and password
 	user, err := mongo.FindUserByEmail(reqBody.Email)
 	if err != nil {
+		log.WithError(err).Errorf("Failed to lookup user: %s", reqBody.Email)
 		renderTempl(c, templates.LoginPage(err.Error()))
 		return
 	}
 
 	if user == nil {
+		log.Warnf("No user was found for: %s", reqBody.Email)
 		renderTempl(c, templates.LoginPage(fmt.Sprintf("No user found for %s", reqBody.Email)))
 		return
 	}
 
 	if err := bcrypt.CompareHashAndPassword([]byte(user.PassowrdHash), []byte(reqBody.Password)); err != nil {
-		renderTempl(c, templates.LoginPage("Email and password are incorrect"))
+		log.Warnf("Password does not match for user: %s", reqBody.Email)
+		renderTempl(c, templates.LoginPage("Email or password are incorrect"))
 		return
 	}
 
@@ -128,7 +140,7 @@ func LoginHandler(c *gin.Context) {
 		},
 	)
 
-	jwtStr, err := token.SignedString(conf.JwtSecret)
+	jwtStr, err := token.SignedString([]byte(conf.JwtSecret))
 	if err != nil {
 		renderTempl(c, templates.LoginPage("An error occured. Please try again later"))
 	}

ui/controllers/controllers.go

@@ -5,6 +5,7 @@ import (
 	"git.preston-baxter.com/Preston_PLB/capstone/frontend-service/db"
 	"git.preston-baxter.com/Preston_PLB/capstone/frontend-service/middleware"
 	"github.com/gin-gonic/gin"
+	"github.com/sirupsen/logrus"
 )
 
 var mongo *db.DB
@@ -19,6 +20,11 @@ func BuildRouter(r *gin.Engine) {
 		panic(err)
 	}
 
+	log = logrus.New()
+	log.SetFormatter(&logrus.TextFormatter{
+		ForceColors:               true,
+	})
+
 	r.GET("/", middleware.AuthMiddleware(false) ,LandingPage)
 	r.GET("/login", middleware.AuthMiddleware(false), LoginPage)
 	r.GET("/signup", middleware.AuthMiddleware(false), SignUpPage)

ui/db/models/models.go

@@ -2,8 +2,8 @@ package models
 
 import "time"
 
-type model struct {
+type CommonFields struct {
-	EntityType string `bson:"ent,omitempty"`
+	EntityType string    `bson:"ent,omitempty"`
-	CreatedAt time.Time `bson:"created_at,omitempty"`
+	CreatedAt  time.Time `bson:"created_at,omitempty"`
-	UpdatedAt time.Time `bson:"updated_at,omitempty"`
+	UpdatedAt  time.Time `bson:"updated_at,omitempty"`
 }

ui/db/models/user.go

@@ -16,7 +16,7 @@ import (
 const USER_TYPE string = "user"
 
 type User struct {
-	*model
+	CommonFields `bson:"obj_info"`
 	mongoId      primitive.ObjectID `bson:"_id,omitempty"`
 	UserId       string             `bson:"user_id,omitempty"`
 	Email        string             `bson:"email,omitempty"`
@@ -28,18 +28,16 @@ func (user *User) Save(client *mongo.Client) error {
 
 	if user.mongoId.IsZero() {
 		now := time.Now()
-		user.model = &model{
+		user.EntityType = USER_TYPE
-			EntityType: USER_TYPE,
+		user.CreatedAt = now
-			CreatedAt: now,
+		user.UpdatedAt = now
-			UpdatedAt: now,
-		}
 		user.UserId = uuid.New().String()
 		user.mongoId = primitive.NewObjectIDFromTimestamp(now)
 	}
 
 	opts := options.Update().SetUpsert(true)
 
-	res, err := client.Database(conf.Mongo.EntDb).Collection(conf.Mongo.EntCol).UpdateOne(context.Background(), user, opts)
+	res, err := client.Database(conf.Mongo.EntDb).Collection(conf.Mongo.EntCol).UpdateOne(context.Background(),  bson.M{"user_id": user.UserId}, bson.M{"$set": user},opts)
 	if err != nil {
 		return err
 	}

ui/db/user.go

@@ -58,7 +58,7 @@ func (db *DB) FindAllUsers() ([]models.User, error) {
 	conf := config.Config()
 
 	opts := options.Find()
-	res, err := db.client.Database(conf.Mongo.EntDb).Collection(conf.Mongo.EntCol).Find(context.Background(), bson.M{"ent": models.USER_TYPE}, opts)
+	res, err := db.client.Database(conf.Mongo.EntDb).Collection(conf.Mongo.EntCol).Find(context.Background(), bson.M{"obj_info.ent": models.USER_TYPE}, opts)
 
 	if err != nil {
 		if err == mongo.ErrNoDocuments {