mirrors/mostlymatter
1
0
Форкнуть 0
Код Задачи Запросы на слияние Действия Пакеты Проекты Релизы Вики Активность

MM-53739: replace store.NewErrNotFound with errors.Wrap in category queries (#30712)

Просмотр исходного кода 
It was a mistake to return ANY error as NewErrNotFound. Because this
can be a DB timeout, or any other network error.

Returning NewErrNotFound would categorize it as 404, eventually printing
the error in the DEBUG level. Changing it to normal error fixes this.

https://mattermost.atlassian.net/browse/MM-63739

```release-note
NONE
```
Этот коммит содержится в:
Agniva De Sarker
2025-04-14 21:11:59 +05:30
коммит произвёл GitHub
родитель ae971225a5
Коммит 848bac2bae
1 изменённых файлов: 4 добавлений и 4 удалений
Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

8
server/channels/store/sqlstore/channel_store_categories.go
Просмотреть файл

@@ -485,7 +485,7 @@ func (s SqlChannelStore) completePopulatingCategoryChannelsT(db dbSelecter, cate
} }
if err := db.Select(&channels, sql, args...); err != nil { if err := db.Select(&channels, sql, args...); err != nil {
return nil, store.NewErrNotFound("ChannelMembers", "<too many fields>").Wrap(err) return nil, errors.Wrap(err, "failed to get channel members")
} }
category.Channels = append(channels, category.Channels...) category.Channels = append(channels, category.Channels...)
@@ -506,7 +506,7 @@ func (s SqlChannelStore) GetSidebarCategory(categoryId string) (*model.SidebarCa
categories := []*sidebarCategoryForJoin{} categories := []*sidebarCategoryForJoin{}
if err = s.GetReplica().Select(&categories, sql, args...); err != nil { if err = s.GetReplica().Select(&categories, sql, args...); err != nil {
return nil, store.NewErrNotFound("SidebarCategories", categoryId).Wrap(err) return nil, errors.Wrap(err, fmt.Sprintf("failed to get category with id=%s", categoryId))
} }
if len(categories) == 0 { if len(categories) == 0 {
@@ -563,7 +563,7 @@ func (s SqlChannelStore) getSidebarCategoriesT(db dbSelecter, userId string, opt
} }
if err := db.Select(&categories, sql, args...); err != nil { if err := db.Select(&categories, sql, args...); err != nil {
return nil, store.NewErrNotFound("SidebarCategories", fmt.Sprintf("userId=%s,teamId=%s", userId, opts.TeamID)).Wrap(err) return nil, errors.Wrap(err, fmt.Sprintf("failed to get categories for userId=%s, teamId=%s", userId, opts.TeamID))
} }
for _, category := range categories { for _, category := range categories {
@@ -624,7 +624,7 @@ func (s SqlChannelStore) GetSidebarCategoryOrder(userId, teamId string) ([]strin
} }
if err := s.GetReplica().Select(&ids, sql, args...); err != nil { if err := s.GetReplica().Select(&ids, sql, args...); err != nil {
return nil, store.NewErrNotFound("SidebarCategories", fmt.Sprintf("userId=%s,teamId=%s", userId, teamId)).Wrap(err) return nil, errors.Wrap(err, fmt.Sprintf("failed to get category order for userId=%s, teamId=%s", userId, teamId))
} }
return ids, nil return ids, nil