GitHub/015
1
0
Fork 0
mirror of https://github.com/keven1024/015.git synced 2026-05-26 07:08:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(backend): improve error handling in DownloadShare by consolidating token validation and share info retrieval

Browse Source
This commit is contained in:
keven1024
2026-04-06 12:08:23 +08:00
parent 499b931c04
commit a1808a64cc
1 changed files with 5 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
backend/internal/controllers/download.go
View File

@@ -28,17 +28,13 @@ func DownloadShare(c *echo.Context) error {
t, err := jwt.ParseWithClaims(token, &claims, func(token *jwt.Token) (interface{}, error) {
return []byte(u.GetEnv("share.download_secret")), nil
})
if err != nil {
return utils.HTTPErrorHandler(c, err)
if err != nil || !t.Valid {
return utils.HTTPErrorHandler(c, lo.Ternary(err != nil, err, ErrInvalidRequest))
}
if !t.Valid {
return utils.HTTPErrorHandler(c, ErrInvalidRequest)
shareInfo, err := models.GetRedisShareInfo(claims.ShareId)
if err != nil || shareInfo == nil {
return utils.HTTPErrorHandler(c, lo.Ternary(err != nil, err, ErrShareNotFound))
}
shareInfo, _ := models.GetRedisShareInfo(claims.ShareId)
if shareInfo == nil {
return utils.HTTPErrorHandler(c, ErrShareNotFound)
}
if shareInfo.Type == models.ShareTypeFile {
fileInfo, _ := models.GetRedisFileInfo(shareInfo.Data)
uploadPath, err := u.GetUploadDirPath()