Ревью

[talantbekov123]

Резюме:

Вы действительно хорошо поработали! Прежде чем ваш проект будет принят, необходимо сделать несколько вещей. Пожалуйста, обратите внимание на скрины ниже, я приложил скриншоты под комментариями, где это необходимо. Надеюсь, их будет легко исправить.

Примечание: я могу прикрепить только один скрин, но вам нужно найти все похожие случаи внутри проекта и исправить их для получения баллов.

Прошу создать и отправить ссылку на пуллреквест, мне придется отклонить работу без проверки если не будет пуллреквеста.

Backend

Критерии, влияющие на работоспособность

• [x] В репозитории есть все необходимые инфраструктурные файлы:
  • [x] в проекте есть package.json;
  • [x] .editorconfig;
  • [x] .eslintrc, расширяющий конфигурацию airbnb-base, а также необходимые для работы линтера dev-зависимости;
  • [x] .gitignore.
  • [x] в .eslintrc добавлено исключение для _id;
• [x] При выполнении команды npx eslint . ошибки линтинга должны отсутствовать.
• [x] В разделе scripts файла package.json:
  • [x] есть команда npm run start, которая запускает сервер на localhost:3000;
  • [x] и npm run dev, которая запускает сервер на localhost:3000 с хот релоудом.
• [x] Когда все зависимости установлены, приложение запускается командой npm run dev без ошибок.
• [ ] Описанные роуты работают корректно:
  • [x] запрос на GET /users/me возвращает информацию о пользователе (email и имя);
  • [x] PATCH /users/me — обновляет информацию о пользователе;
  • [x] GET /movies — все сохранённые пользователем фильмы;
  • [x] POST /movies — создаёт фильм с переданными в теле данными;
  • [ ] DELETE /movies/movieId — удаляет сохранённый фильмы по _id;

    https://snipboard.io/Z485Mw.jpg Если пользователя или карточки с запрошенным идентификатором нет, API должен возвращать объект с ключом message и статус 404; При условии что айди валидный.

  • [x] POST /signup — создаёт пользователя с переданными в теле данными;
  • [x] POST /signin — возвращает JWT, если в теле запроса переданы правильные почта и пароль.
  • [x] Если вы сохраняете JWT в куках, роут /signout должен удалять JWT из куки.
• [x] Все роуты, кроме /signin и /signup, защищены авторизацией.
• [x] Роуты пользователей и роуты фильмов описаны в отдельных файлах.
• [x] Ошибки API обрабатываются:
  • [x] если в запросе что-то не так, сервер возвращает ответ с ошибкой и её статусом;
  • [x] асинхронные обработчики завершены блоком catch;
  • [x] API не возвращает стандартных ошибок базы данных или Node.js.
• [x] В production-режиме адрес базы данных берётся из process.env.
• [x] Реализовано бережное хранение пароля:
  • [x] пароль хранится в зашифрованном виде;
  • [x] API не возвращает хеш пароля клиенту.
• [x] Данные валидируются перед добавлением в базу.
• [x] Пользователь не может удалить сохранённую карточку из профиля другого пользователя.
• [x] К серверу можно обратиться по https по адресу домена, указанному в README.md.
• [ ] Правильно реализовано хранение секретного ключа для создания JWT:

  https://snipboard.io/KloMRc.jpg достаточно указать переменную, не нужно оборачивать в ${}

  • [x] для продакшн сборки он хранится в .env файле — этот файл не добавляется в git;
  • [x] в режиме разработки (при process.env.NODE_ENV !== 'production') код запускается и работает и без .env файла.

Хорошие практики

• [x] Асинхронные операции реализованы промисами или async/await.
• [x] Запрос валидируется перед передачей контроллеру. Тело и, где необходимо, заголовки и параметры, проверяются по соответствующим схемам. Если запрос не соответствует схеме, обработка не передаётся контроллеру, и клиент получает ошибку валидации.
• [ ] Настроено логгирование:
  • [ ] запросы и ответы записываются в файл request.log;

    https://snipboard.io/TuiZQX.jpg limiter подключен перед логгером запросов. Запросы, отклоненные лимитером, не будут добавлены в лог запросов. Необходимо первым в цепочке подключить логгер, а затем limiter

  • [x] ошибки записываются в файл error.log;
  • [x] файлы логов не добавляются в репозиторий.
• [x] Ошибки обрабатываются централизованным обработчиком.
• [x] Централизованный обработчик ошибок описан в отдельном модуле.
• [x] В случае ошибки API возвращает статус ответа, который соответствует типу ошибки.
• [x] К серверу можно обратиться по https по адресу домена, указанному в README.md.
• [x] API приложения располагается на домене вида: name.zone/api, а не просто name.zone
• [x] Все роуты подключены в файле index.js, который находится в папке routes. Оттуда единый роут подключается в файле app.js.

Рекомендации

• [x] Для ошибок API созданы классы, расширяющие конструктор Error.
• [x] Используется модуль Helmet для установки заголовков, связанных с безопасностью.
• [ ] Конфигурация и константы в отдельных файлах:
  • [x] адрес Mongo-сервера в режиме разработки вынесен в отдельный конфигурационный файл;
  • [ ] константы приложения — сообщения ответов и ошибок — вынесены в отдельный файл с константами.

    https://snipboard.io/708dyt.jpg

• [x] Настроен rate limiter: число запросов с одного IP в единицу времени ограничено.
• [ ] rate limiter сконфигурирован в отдельном файле и импортируется в app.js

  https://snipboard.io/0RbZUz.jpg

• [x] API размещено на отдельном поддомене, например: api.news-explorer.tk.

Количество баллов: 68

[talantbekov123]

Резюме:

Отличная работа, все комментарии исправлены! Похвально! Принимаю.

Backend

Критерии, влияющие на работоспособность

• [x] В репозитории есть все необходимые инфраструктурные файлы:
  • [x] в проекте есть package.json;
  • [x] .editorconfig;
  • [x] .eslintrc, расширяющий конфигурацию airbnb-base, а также необходимые для работы линтера dev-зависимости;
  • [x] .gitignore.
  • [x] в .eslintrc добавлено исключение для _id;
• [ ] При выполнении команды npx eslint . ошибки линтинга должны отсутствовать.

  https://snipboard.io/h35I7b.jpg

• [x] В разделе scripts файла package.json:
  • [x] есть команда npm run start, которая запускает сервер на localhost:3000;
  • [x] и npm run dev, которая запускает сервер на localhost:3000 с хот релоудом.
• [x] Когда все зависимости установлены, приложение запускается командой npm run dev без ошибок.
• [x] Описанные роуты работают корректно:
  • [x] запрос на GET /users/me возвращает информацию о пользователе (email и имя);
  • [x] PATCH /users/me — обновляет информацию о пользователе;
  • [x] GET /movies — все сохранённые пользователем фильмы;
  • [x] POST /movies — создаёт фильм с переданными в теле данными;
  • [x] DELETE /movies/movieId — удаляет сохранённый фильмы по _id;
  • [x] POST /signup — создаёт пользователя с переданными в теле данными;
  • [x] POST /signin — возвращает JWT, если в теле запроса переданы правильные почта и пароль.
  • [x] Если вы сохраняете JWT в куках, роут /signout должен удалять JWT из куки.
• [x] Все роуты, кроме /signin и /signup, защищены авторизацией.
• [x] Роуты пользователей и роуты фильмов описаны в отдельных файлах.
• [x] Ошибки API обрабатываются:
  • [x] если в запросе что-то не так, сервер возвращает ответ с ошибкой и её статусом;
  • [x] асинхронные обработчики завершены блоком catch;
  • [x] API не возвращает стандартных ошибок базы данных или Node.js.
• [x] В production-режиме адрес базы данных берётся из process.env.
• [x] Реализовано бережное хранение пароля:
  • [x] пароль хранится в зашифрованном виде;
  • [x] API не возвращает хеш пароля клиенту.
• [x] Данные валидируются перед добавлением в базу.
• [x] Пользователь не может удалить сохранённую карточку из профиля другого пользователя.
• [x] К серверу можно обратиться по https по адресу домена, указанному в README.md.
• [x] Правильно реализовано хранение секретного ключа для создания JWT:

  https://snipboard.io/KloMRc.jpg достаточно указать переменную, не нужно оборачивать в ${}

  • [x] для продакшн сборки он хранится в .env файле — этот файл не добавляется в git;
  • [x] в режиме разработки (при process.env.NODE_ENV !== 'production') код запускается и работает и без .env файла.

Хорошие практики

• [x] Асинхронные операции реализованы промисами или async/await.
• [x] Запрос валидируется перед передачей контроллеру. Тело и, где необходимо, заголовки и параметры, проверяются по соответствующим схемам. Если запрос не соответствует схеме, обработка не передаётся контроллеру, и клиент получает ошибку валидации.
• [x] Настроено логгирование:
  • [x] запросы и ответы записываются в файл request.log;
  • [x] ошибки записываются в файл error.log;
  • [x] файлы логов не добавляются в репозиторий.
• [x] Ошибки обрабатываются централизованным обработчиком.
• [x] Централизованный обработчик ошибок описан в отдельном модуле.
• [x] В случае ошибки API возвращает статус ответа, который соответствует типу ошибки.
• [x] К серверу можно обратиться по https по адресу домена, указанному в README.md.
• [x] API приложения располагается на домене вида: name.zone/api, а не просто name.zone
• [x] Все роуты подключены в файле index.js, который находится в папке routes. Оттуда единый роут подключается в файле app.js.

Рекомендации

• [x] Для ошибок API созданы классы, расширяющие конструктор Error.
• [x] Используется модуль Helmet для установки заголовков, связанных с безопасностью.
• [x] Конфигурация и константы в отдельных файлах:
  • [x] адрес Mongo-сервера в режиме разработки вынесен в отдельный конфигурационный файл;
  • [x] константы приложения — сообщения ответов и ошибок — вынесены в отдельный файл с константами.
• [x] Настроен rate limiter: число запросов с одного IP в единицу времени ограничено.
• [x] rate limiter сконфигурирован в отдельном файле и импортируется в app.js
• [x] API размещено на отдельном поддомене, например: api.news-explorer.tk.

Количество баллов: 97