Compare commits
5 Commits
v1.0.16
...
f8e7b01a28
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
f8e7b01a28 | ||
|
|
332a84aa57 | ||
|
|
e9fdea664f | ||
|
|
7704fa9ecb | ||
|
|
9a0019f3ad |
@@ -49,34 +49,21 @@ func (aH *AccessHandler) Login(c *gin.Context) {
|
||||
aH.logger.Error("Login", "user empty")
|
||||
c.JSON(http.StatusBadRequest, models.NewJsonMessageResponse("user empty"))
|
||||
return
|
||||
} else if !user.ExpirationIsValid() {
|
||||
aH.logger.Error("Login", fmt.Sprintf("user %s is expired", user.Name))
|
||||
c.JSON(http.StatusUnauthorized, models.NewJsonMessageResponse("user "+user.Name+" is expired"))
|
||||
return
|
||||
}
|
||||
|
||||
// Fetch user record from DB
|
||||
var dbRecord []models.User
|
||||
err := aH.dbHandler.GetByKey(&dbRecord, "user_name", user.Name, false)
|
||||
if err != nil {
|
||||
aH.logger.Error("Login", err)
|
||||
c.JSON(http.StatusBadRequest, models.NewJsonErrorResponse(err))
|
||||
return
|
||||
}
|
||||
|
||||
if len(dbRecord) > 1 {
|
||||
aH.logger.Error("Login", "more than one record found")
|
||||
c.JSON(http.StatusInternalServerError, models.NewJsonMessageResponse("internal error"))
|
||||
dbUser, hasError := aH.getUserFromDB(c, user.Name)
|
||||
if hasError {
|
||||
return
|
||||
}
|
||||
|
||||
// Check password
|
||||
if !utils.CheckPassword(user.Password, dbRecord[0].Password) {
|
||||
if !utils.CheckPassword(user.Password, dbUser.Password) {
|
||||
aH.logger.Error("Login", "invalid password")
|
||||
c.JSON(http.StatusUnauthorized, models.NewJsonMessageResponse("invalid credentials"))
|
||||
return
|
||||
}
|
||||
user = dbRecord[0]
|
||||
user = dbUser
|
||||
|
||||
// -----------------------------
|
||||
// 🔑 TOKEN CREATION
|
||||
@@ -93,7 +80,6 @@ func (aH *AccessHandler) Login(c *gin.Context) {
|
||||
"role": user.Role,
|
||||
"type": "access",
|
||||
"exp": accessTokenExp.Unix(),
|
||||
"userExpiration": user.Expiration,
|
||||
})
|
||||
|
||||
// Create refresh token
|
||||
@@ -103,7 +89,6 @@ func (aH *AccessHandler) Login(c *gin.Context) {
|
||||
"role": user.Role,
|
||||
"type": "refresh",
|
||||
"exp": refreshTokenExp.Unix(),
|
||||
"userExpiration": user.GetExpiration(),
|
||||
})
|
||||
|
||||
// Sign tokens
|
||||
@@ -179,11 +164,9 @@ func (aH *AccessHandler) Refresh(c *gin.Context) {
|
||||
|
||||
username := claims["username"].(string)
|
||||
id := int(claims["id"].(float64))
|
||||
role := claims["role"].(string)
|
||||
|
||||
if !expirationDateValid(claims["userExpiration"].(string)) {
|
||||
aH.logger.Error("Login", fmt.Sprintf("user %s is expired", username))
|
||||
c.JSON(http.StatusUnauthorized, models.NewJsonMessageResponse("user "+username+" is expired"))
|
||||
user, hasError := aH.getUserFromDB(c, username)
|
||||
if hasError {
|
||||
return
|
||||
}
|
||||
|
||||
@@ -193,7 +176,7 @@ func (aH *AccessHandler) Refresh(c *gin.Context) {
|
||||
newAccess := jwt.NewWithClaims(jwt.SigningMethodHS256, jwt.MapClaims{
|
||||
"id": id,
|
||||
"username": username,
|
||||
"role": role,
|
||||
"role": user.Role,
|
||||
"exp": accessExp.Unix(),
|
||||
})
|
||||
accessString, _ := newAccess.SignedString(ACCESS_SECRET)
|
||||
@@ -253,15 +236,33 @@ func (aH *AccessHandler) Logout(c *gin.Context) {
|
||||
c.JSON(http.StatusOK, gin.H{"message": "logged out"})
|
||||
}
|
||||
|
||||
func expirationDateValid(expiration string) bool {
|
||||
if expiration == "" {
|
||||
return true // No expiration = always valid
|
||||
}
|
||||
|
||||
t, err := time.Parse(time.RFC3339, expiration)
|
||||
func (aH *AccessHandler) getUserFromDB(c *gin.Context, userName string) (user models.User, hasError bool) {
|
||||
hasError = true
|
||||
// Fetch user record from DB
|
||||
var dbRecord []models.User
|
||||
err := aH.dbHandler.GetByKey(&dbRecord, "user_name", userName, false)
|
||||
if err != nil {
|
||||
return false // Invalid date format
|
||||
aH.logger.Error("Login", err)
|
||||
c.JSON(http.StatusBadRequest, models.NewJsonErrorResponse(err))
|
||||
return
|
||||
}
|
||||
|
||||
return time.Now().Before(t)
|
||||
if len(dbRecord) == 0 {
|
||||
aH.logger.Error("Login", "no user "+userName+" found")
|
||||
c.JSON(http.StatusUnauthorized, models.NewJsonMessageResponse("invalid credentials"))
|
||||
return
|
||||
}
|
||||
|
||||
if len(dbRecord) > 1 {
|
||||
aH.logger.Error("Login", "more than one record found")
|
||||
c.JSON(http.StatusInternalServerError, models.NewJsonMessageResponse("internal error"))
|
||||
return
|
||||
}
|
||||
|
||||
if !dbRecord[0].ExpirationIsValid() {
|
||||
aH.logger.Error("Login", fmt.Sprintf("user %s is expired", userName))
|
||||
c.JSON(http.StatusUnauthorized, models.NewJsonMessageResponse("user "+userName+" is expired"))
|
||||
return
|
||||
}
|
||||
return dbRecord[0], false
|
||||
}
|
||||
|
||||
@@ -12,7 +12,7 @@ func (r *Permissions) DefaultPermissions() {
|
||||
*r = append(*r,
|
||||
Permission{Name: "settings", Permission: 7},
|
||||
Permission{Name: "userSettings", Permission: 7},
|
||||
Permission{Name: "members", Permission: 7},
|
||||
Permission{Name: "members", Permission: 15},
|
||||
Permission{Name: "events", Permission: 7},
|
||||
Permission{Name: "responsible", Permission: 7},
|
||||
)
|
||||
|
||||
@@ -12,6 +12,7 @@ type Settings struct {
|
||||
SecondaryColor string `json:"secondaryColor,omitempty"`
|
||||
SecondaryColorText string `json:"secondaryColorText,omitempty"`
|
||||
Icon string `json:"icon,omitempty"`
|
||||
AppName string `json:"appName,omitempty"`
|
||||
DatabaseName string `json:"databaseName,omitempty"`
|
||||
DatabaseToken string `json:"databaseToken,omitempty"`
|
||||
}
|
||||
|
||||
@@ -1,6 +1,8 @@
|
||||
package models
|
||||
|
||||
import "time"
|
||||
import (
|
||||
"time"
|
||||
)
|
||||
|
||||
type User struct {
|
||||
Id uint `gorm:"primaryKey" json:"id"`
|
||||
@@ -8,7 +10,7 @@ type User struct {
|
||||
Email string `gorm:"column:email" json:"email"`
|
||||
Role string `gorm:"column:role" json:"role,omitempty"`
|
||||
Password string `gorm:"column:password" json:"password"`
|
||||
Expiration *time.Time `gorm:"column:expiration" json:"expiration,omitempty"`
|
||||
Expiration string `gorm:"column:expiration" json:"expiration,omitempty"`
|
||||
Settings Settings `gorm:"type:json" json:"settings"`
|
||||
}
|
||||
|
||||
@@ -17,15 +19,10 @@ func (u *User) IsValid() bool {
|
||||
}
|
||||
|
||||
func (u *User) ExpirationIsValid() bool {
|
||||
if u.Expiration == nil {
|
||||
if u.Expiration == "" || u.Expiration == "never" {
|
||||
return true
|
||||
}
|
||||
return time.Now().Before(*u.Expiration)
|
||||
}
|
||||
|
||||
func (u *User) GetExpiration() string {
|
||||
if u.Expiration == nil {
|
||||
return ""
|
||||
}
|
||||
return u.Expiration.Format(time.RFC3339)
|
||||
loc := time.Now().Location()
|
||||
parsedTime, _ := time.ParseInLocation("2006-01-02 15:04:05", u.Expiration, loc)
|
||||
return parsedTime.After(time.Now())
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user