Почему важно проверять response.ok в Fetch API и почему HTTP-ошибки не вызывают отклонение промисов

[yeezussniper]

А еще некоторые индивиды возвращают 200, при этом отображая код ошибки внутри самого тела запроса.

{
 code: 404,
 data: {}
}

Был у нас такой прикол с интеграцией стороннего сервиса.

В таком случае приходится еще и на code проверять 😂

[olivera507224]

Ну, технически никто не запрещает, имеют право. Главное чтобы это было хорошо задокументировано.

[Zukomux]

Руки рубит за такое. Периодически встречаю такие подходы и удивляюсь! Возвращать ОК200 с кодом ошибки зачем? Чтобы что? Есть определенный стандарт ошибок, которые закрывают 99.9% случаев. В результате вместо нормальной обработки ошибок ты пишешь двойной парсер с лапшой

[danilanemaster]

Ну, иногда полезно отличать отсутствие запрошеного объекта от временной пропажи сервера

[Free_ze]

Для этого семантичнее было бы возвращать 503 (Service Unavailable), т.к. статусы 4xx сигналят про ошибку клиента, а не временную проблему сервера. С точки зрения HTTP, раз за разом повторять запрос, вернувший 4хх в надежде на изменение - это безумие.

[Free_ze]

*Разумеется, речь про настоящий код ответа, а не про эрзац в теле.

[ooko]

Если http это только транспорт то все правильно. А транспортом может быть к примеру и вебсокет

И кстати если дублировать код в http то уже на фронте приходится код дополнительно проверять - так как кодом может ответить какая-то прокся

[Free_ze]

Статус-коды HTTP в сообщении на веб-сокетах - это сюр.

[ooko]

Где вы увидели что написал про коды-http в веб-сокетах? Есть ошибки которые возвращает сервер приложения, они не обязательно должны быть совместимы с кодами-http. HTTP используется как транспорт, также как TCP/IP для HTTP. Вы же не используете коды ошибок TCP/IP в HTTP

[Free_ze]

Вы назвали полностью правильным пример, в котором делаются именно так.

[kspshnik]

По уму надо не просто какую-то фихню строковую вернуть, а то, что прислал сервер. Но добавить туда код ответа:

if (res.ok) {
 return res.json();
}
 return res.json()
   .then((error) => {
 // Добавляем свойство с заведомо не имеющимся в ответе сервера именем для кода ошибки
      error.httpResponseCode = res.status;
      return Promise.reject(error);
}