Skip to content

review 2#2

Open
bjlag wants to merge 94 commits intoreviewfrom
main
Open

review 2#2
bjlag wants to merge 94 commits intoreviewfrom
main

Conversation

@bjlag
Copy link
Owner

@bjlag bjlag commented Mar 19, 2025

В первой половине было сделано:

  • Авторизация (jwt access, refresh токены).
  • Сервер хранит данные в зашифрованном виде.
  • Протокол обмена gRPC.
  • Клиент расшифровывает полученные данные с сервера и кладет в локальную базу.
  • UI на базе github.com/charmbracelet/bubbles.

Доделал:

  • Поддержал все типы приватных данных в UI: пароли, текст, банковские карты и бинарники.
  • Синхронизация с сервером. Не стал усложнять, источник правды всегда сервер. Если версия на клиенте отличается от версии на сервере, то клиент предлагает синхронизироваться.
  • Задокументировал код.
  • Написал функциональные тесты. Не стал тратить время на юниты, с ними все понятно. А вот с функциональными хотел разобраться. Поднимается база в отдельном Docker контейнере, накатываются миграции, применяются фикстуры и после запускаюстя тесты. Покрыт тестами RPC ручки.
  • Cборки для разных платформ.
  • Вывод версии клиента и даты билда, команда с флагом version, билдится командой make build-client.
  • При разлогине данные в локальной базе шифруются, при логине восстанавливаются.
  • Под каждого пользователя, который успешно авторизовался, создается своя локальная база.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant