feat(api tokens): better error message for invalid tokens
This commit is contained in:
parent
e295d75e6e
commit
8f3d18a809
@ -23,7 +23,6 @@ import (
|
|||||||
|
|
||||||
"code.vikunja.io/api/pkg/config"
|
"code.vikunja.io/api/pkg/config"
|
||||||
"code.vikunja.io/api/pkg/db"
|
"code.vikunja.io/api/pkg/db"
|
||||||
"code.vikunja.io/api/pkg/log"
|
|
||||||
"code.vikunja.io/api/pkg/models"
|
"code.vikunja.io/api/pkg/models"
|
||||||
|
|
||||||
echojwt "github.com/labstack/echo-jwt/v4"
|
echojwt "github.com/labstack/echo-jwt/v4"
|
||||||
@ -43,7 +42,6 @@ func SetupTokenMiddleware() echo.MiddlewareFunc {
|
|||||||
if strings.HasPrefix(s, "Bearer "+models.APITokenPrefix) {
|
if strings.HasPrefix(s, "Bearer "+models.APITokenPrefix) {
|
||||||
err := checkAPITokenAndPutItInContext(s, c)
|
err := checkAPITokenAndPutItInContext(s, c)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
log.Errorf("Could not check api token: %v", err)
|
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
return true
|
return true
|
||||||
@ -52,6 +50,13 @@ func SetupTokenMiddleware() echo.MiddlewareFunc {
|
|||||||
|
|
||||||
return false
|
return false
|
||||||
},
|
},
|
||||||
|
ErrorHandler: func(c echo.Context, err error) error {
|
||||||
|
if err != nil {
|
||||||
|
return echo.NewHTTPError(http.StatusUnauthorized, "missing, malformed, expired or otherwise invalid token provided").SetInternal(err)
|
||||||
|
}
|
||||||
|
|
||||||
|
return nil
|
||||||
|
},
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user