go-park-mail-ru/2019_1_newteam2

2019_1_newteam2's Issues

Тесты

sergeychur

• UsersPaginate
• UpdateUser
• GetUser
• DeleteUser

tsaanstu

• SignUpAPI
• Logout
• IsLogin (true)
• IsLogin (false)
• LoginAPI

РК 1

• не хранить картинки в репе

• использовать пакет log для логов (или что-то подобное)

• cookie.HttpOnly = false - плохо

• "session_id" либо в константу, либо в конфиг

• user_api.go#L123 - выпилить

• user_api.go#L168 - return

• непонятно как и зачем функция CreateUser

• использовать crypto/bcrypt для шифрования паролей

• ⚠️не хранить пароль в jwt

• не работают тесты

• coverage: 47.8% of statements

go test -v ./... -cover -coverprofile=cover.out
go tool cover -html=cover.out -o cover.html

newRK1

• не хранить картинки в репе

• использовать пакет log для логов (или что-то подобное)

• cookie.HttpOnly = false - плохо

• "session_id" либо в константу, либо в конфиг

• user_api.go#L123 - выпилить

• user_api.go#L168 - return

• непонятно как и зачем функция CreateUser

• использовать crypto/bcrypt для шифрования паролей

• ⚠️не хранить пароль в jwt

• не работают тесты

• coverage: 47.8% of statements

go test -v ./... -cover -coverprofile=cover.out
go tool cover -html=cover.out -o cover.html

РК 3

• (tsaanstu) В вашу игру можно поиграть в мультиплеере. Должна быть реализована многопользовательская игровая логика и интеграция с фронтендом.

• (tsaanstu) В вашем проекте есть микросервисы, которые общаются друг с другом по grpc. Игра и авторизация должны быть отдельными сервисами.

• К вашему проекту прикручен мониторинг через prometheus и можно посмотреть графики по вашим сервисам в grafana. Необходимый минимум графиков: хиты на API с разбивкой по статусам ответов, количество активных комнат/игроков в играх.

• (tsaanstu) Prometheus

• (tsaanstu) Grafana

• (sergeychur) В ваших приложениях нет захардкоженных урлов/логинов/паролей. Все хранится в конфиге, передаётся в переменной окружения или хранится в vault (не обязательно)

• Проект покрыт тестами минимум на 65%

---

• (tsaanstu) Результаты из игры переносятся в общую таблицу (отдельный микросервис)

• (sergeychur) Скрипт запуска

• (sergeychur) Вынести в конфиг количество игроков в комнатах и добавить порт нового микросервиса

К РК1

• REST (tsaanstu)
• JWT (tsaanstu)
• Модули (sergeychur)
• Тесты