feat(terminate_all_session): Terminate all user sessions #35
Conversation
| @@ -14,7 +14,7 @@ class Settings(BaseSettings): | |||
| listen_port: int = 8000 | |||
| allowed_hosts: list = const.DEFAULT_ALLOWED_HOSTS | |||
|
|
|||
| redis_host: str = "redis" | |||
| redis_host: str = "localhost" | |||
There was a problem hiding this comment.
Тут лучше оставить имя контейнера по умолчанию
There was a problem hiding this comment.
имхо, когда будем собирать докер - это может усложнить работу, лучше 'redis'
то же самое с postgres, но я могу это поправить в ветке с докером
| @@ -14,7 +14,7 @@ class Settings(BaseSettings): | |||
| listen_port: int = 8000 | |||
| allowed_hosts: list = const.DEFAULT_ALLOWED_HOSTS | |||
|
|
|||
| redis_host: str = "redis" | |||
| redis_host: str = "localhost" | |||
There was a problem hiding this comment.
имхо, когда будем собирать докер - это может усложнить работу, лучше 'redis'
то же самое с postgres, но я могу это поправить в ветке с докером
| return await conn.delete(signature_key) | ||
|
|
||
| async def delete_all(self, user_id: UUID4, count_size: int = 10) -> int: | ||
| pattern = f"{self.namespace}:{user_id}:*" |
There was a problem hiding this comment.
возможно я не понимаю как клиент редиса работает, но если мы создаем только пары ключ-значение вида f"{self.namespace}:{user_id}": signature то зачем нам такой паттерн поиска
f"{self.namespace}:{user_id}:*"
?
- кажется что будет всего один такой ключ - у нас у каждого пользователя одна сигнатура
- ключ при сохранении на включает замыкающее ":" + какие-либо еще символы
| @@ -54,3 +54,25 @@ async def logout( | |||
| ) | |||
| return LogoutResponse() | |||
|
|
|||
|
|
|||
There was a problem hiding this comment.
заметил что в logout методе есть access_token из кук, но он никуда дальше не передается
по идее надо по нему верификацию делать как в terminate_all_sessions
Task #7