generated from Sithas/conan_template
DAO - Полное завершение класса
This commit is contained in:
@@ -6,6 +6,14 @@
|
||||
- Посмотреть, что дает MySQL, какие там есть возможность
|
||||
- Посмотреть и подумать, что лучше - корутины или многопоточность?
|
||||
- Покрыть тестами класс User и AuthRegistrationExecutor
|
||||
- Добавить clang-format(через CLion)
|
||||
- Перевести GetByUUID GetByLogin на const ref/string_view в IUserDAO - также не vector, а span(погуглить)
|
||||
- Привести к единоу виду функции IUserDAO
|
||||
- Пройтись по коду и максимально наставить const
|
||||
- Указать возможные исключения в интерфейсах DAO
|
||||
- Вынести User в структуру. Hashed Password структура должна изначально состоять в другой структуре
|
||||
- SharedPtr - передавать по константной ссылке.
|
||||
- Сделать интеграционный тест по ручкам
|
||||
|
||||
# UseCase'ы приложения:
|
||||
|
||||
@@ -111,8 +119,8 @@
|
||||
```
|
||||
|
||||
##### Errors
|
||||
* `401 BAD_CREDENTIALS` — неверный логин/пароль (А1)
|
||||
* `422 VALIDATION_FAILED` — пустые поля (А2)
|
||||
* `401 BAD_CREDENTIALS` — неверный логин/пароль (B1)
|
||||
* `422 VALIDATION_FAILED` — пустые поля (B2)
|
||||
|
||||
#### Logout-Request
|
||||
```
|
||||
|
||||
@@ -31,7 +31,7 @@ bool MemoryAuthDAO::Logout(const std::string& auth_token)
|
||||
{
|
||||
string user_uuid = auth_tokens_to_users_uuids_[auth_token];
|
||||
|
||||
if (!HasAuthorizedUser()) return false;
|
||||
if (!HasAuthorized(auth_token)) return false;
|
||||
|
||||
users_uuids_to_auth_tokens_.erase(user_uuid);
|
||||
auth_tokens_to_users_uuids_.erase(auth_token);
|
||||
|
||||
Reference in New Issue
Block a user