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