EventHub/EventHubBack
3
1
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

P1: Редактирование профиля пользователя (PUT /v1/user/me) #48

New Issue
Closed
opened 2026-07-19 22:02:07 +03:00 by cursor-ai · 3 comments
cursor-ai commented 2026-07-19 22:02:07 +03:00
Owner
Copy Link
Copy Source

Проблема

Клиентский SPA (EventHubFront) уже умеет переключать язык UI (RU/EN) по аналогии с Admin, но у пользователя есть только GET /v1/user/me — API обновления собственного профиля отсутствует.

Сейчас локаль клиента хранится в localStorage и при логине/checkAuth опционально подтягивается из user.language. В Admin язык сохраняется через PUT /v1/admin/me. Без user update API клиент не может персистить язык и другие поля профиля на бэкенде. В EventHubFrontSpec явно зафиксировано: редактирование профиля не реализуется, пока нет user update API.

Влияние

  • Язык интерфейса не синхронизируется между устройствами/сессиями.
  • Страница профиля на клиенте остаётся read-only.
  • Расхождение с Admin: у администраторов обновление профиля уже есть, у обычных пользователей — нет.

Ожидаемый результат

Добавить эндпоинт обновления профиля текущего аутентифицированного пользователя, по аналогии с PUT /v1/admin/me.

Эндпоинт

  • Метод: PUT /v1/user/me (предпочтительно — единообразие с admin; допустим PATCH, если явно обосновано)
  • Auth: обязательный user JWT (handler_utils:auth_user/1); без токена — 401
  • Ответ: 200 + актуальный профиль в том же формате, что GET /v1/user/me (handler_utils:user_to_json/1)

Разрешённые поля тела запроса

Частичное обновление (передаются только изменяемые поля). Разрешить поля, уже присутствующие в модели #user{} / ответе GET и в admin profile update:

Поле Тип Примечание
language string Обязательный минимум для i18n клиента (ru / en или согласованный enum)
nickname string \| null
timezone string \| null
phone string \| null
avatar_url string \| null как у admin
preferences object \| null
social_links array of string \| null опционально; есть в GET-схеме user, нет у admin

Запретить изменение через этот эндпоинт: email, password / password_hash, role, status, reason, id, системные timestamps. Неизвестные поля — игнорировать или 400 (выбрать один подход и задокументировать; у admin неизвестные ключи частично проходят через convert — лучше валидировать whitelist).

Валидация

  • JSON-тело обязательно, невалидный JSON → 400
  • Whitelist полей (см. выше); типы как в logic_user:validate_user_update/1 (+ avatar_url, при необходимости social_links)
  • language: допустимые значения согласовать с клиентом/Admin (как минимум ru, en)
  • Пользователь может обновлять только себя; чужой id в пути нет (ресурс /me)
  • Frozen/deleted: поведение как у остальных user API (не давать писать профиль, если статус не active — если так уже принято)

Реализация (ориентир)

  • Расширить handler_user_me.erl: сейчас только GET → добавить PUT по образцу admin_handler_me.erl
  • Persistence: уже есть core_user:update/2 и set_field для нужных полей; при необходимости — тонкий logic_user:update_me/2 с whitelist (не путать с update_user_admin/2, где можно менять role/status)
  • OpenAPI/trails: описать PUT в handler_user_me:trails/0 → client-swagger.json
  • Спека: после тестов обновить EventHubBackSpec (и при необходимости отметить в FrontSpec, что API появился — follow-up на фронт отдельно)

Тесты

API-тесты на PUT /v1/user/me: успех (language и прочие поля), 401, 400, запрет смены email/role/status, ответ совпадает со схемой GET.

Критерии приёмки

  • Реализован PUT /v1/user/me (или согласованный эквивалент) для аутентифицированного пользователя
  • Можно обновить как минимум language; также nickname, timezone, phone, avatar_url, preferences (и при решении — social_links)
  • Нельзя сменить email, пароль, role, status через этот эндпоинт
  • Без/с невалидным токеном — 401; невалидное тело — 400
  • Успешный ответ 200 возвращает полный профиль как GET
  • OpenAPI/client-swagger и trails обновлены
  • Есть покрывающие API-тесты, проходящие в CI
  • После успешных тестов обновлена бэкенд-спека (EventHubBackSpec)

Контекст / follow-up

  • Триггер: EventHubFront — UI i18n уже есть, персист языка на бэкенде ждёт этот API.
  • Ориентир: PUT /v1/admin/me в admin_handler_me.erl.
  • Задачу на Front (форма профиля + sync locale через PUT) не создавать сейчас; после готовности бэка — отдельный follow-up.

Файлы (подсказка)

  • src/handlers/handler_user_me.erl — сейчас только GET
  • src/handlers/admin/admin_handler_me.erl — образец PUT
  • src/core/core_user.erl — update/2, set_field/3
  • src/logic/logic_user.erl — валидация полей (admin path)
  • src/swagger/client-swagger.json
  • EventHubSpec/EventHubBackSpec.md

Приоритет

P1

## Проблема Клиентский SPA (EventHubFront) уже умеет переключать язык UI (RU/EN) по аналогии с Admin, но у пользователя есть только `GET /v1/user/me` — **API обновления собственного профиля отсутствует**. Сейчас локаль клиента хранится в `localStorage` и при логине/checkAuth опционально подтягивается из `user.language`. В Admin язык сохраняется через `PUT /v1/admin/me`. Без user update API клиент не может персистить язык и другие поля профиля на бэкенде. В `EventHubFrontSpec` явно зафиксировано: редактирование профиля не реализуется, пока нет user update API. ## Влияние - Язык интерфейса не синхронизируется между устройствами/сессиями. - Страница профиля на клиенте остаётся read-only. - Расхождение с Admin: у администраторов обновление профиля уже есть, у обычных пользователей — нет. ## Ожидаемый результат Добавить эндпоинт обновления профиля **текущего аутентифицированного пользователя**, по аналогии с `PUT /v1/admin/me`. ### Эндпоинт - **Метод:** `PUT /v1/user/me` (предпочтительно — единообразие с admin; допустим `PATCH`, если явно обосновано) - **Auth:** обязательный user JWT (`handler_utils:auth_user/1`); без токена — `401` - **Ответ:** `200` + актуальный профиль в том же формате, что `GET /v1/user/me` (`handler_utils:user_to_json/1`) ### Разрешённые поля тела запроса Частичное обновление (передаются только изменяемые поля). Разрешить поля, уже присутствующие в модели `#user{}` / ответе GET и в admin profile update: | Поле | Тип | Примечание | |------|-----|------------| | `language` | string | **Обязательный минимум** для i18n клиента (`ru` / `en` или согласованный enum) | | `nickname` | string \\| null | | | `timezone` | string \\| null | | | `phone` | string \\| null | | | `avatar_url` | string \\| null | как у admin | | `preferences` | object \\| null | | | `social_links` | array of string \\| null | опционально; есть в GET-схеме user, нет у admin | **Запретить** изменение через этот эндпоинт: `email`, `password` / `password_hash`, `role`, `status`, `reason`, `id`, системные timestamps. Неизвестные поля — игнорировать или `400` (выбрать один подход и задокументировать; у admin неизвестные ключи частично проходят через convert — лучше валидировать whitelist). ### Валидация - JSON-тело обязательно, невалидный JSON → `400` - Whitelist полей (см. выше); типы как в `logic_user:validate_user_update/1` (+ `avatar_url`, при необходимости `social_links`) - `language`: допустимые значения согласовать с клиентом/Admin (как минимум `ru`, `en`) - Пользователь может обновлять **только себя**; чужой id в пути нет (ресурс `/me`) - Frozen/deleted: поведение как у остальных user API (не давать писать профиль, если статус не `active` — если так уже принято) ### Реализация (ориентир) - Расширить `handler_user_me.erl`: сейчас только GET → добавить PUT по образцу `admin_handler_me.erl` - Persistence: уже есть `core_user:update/2` и `set_field` для нужных полей; при необходимости — тонкий `logic_user:update_me/2` с whitelist (не путать с `update_user_admin/2`, где можно менять role/status) - OpenAPI/trails: описать PUT в `handler_user_me:trails/0` → `client-swagger.json` - Спека: после тестов обновить `EventHubBackSpec` (и при необходимости отметить в FrontSpec, что API появился — follow-up на фронт отдельно) ### Тесты API-тесты на `PUT /v1/user/me`: успех (language и прочие поля), `401`, `400`, запрет смены email/role/status, ответ совпадает со схемой GET. ## Критерии приёмки - [ ] Реализован `PUT /v1/user/me` (или согласованный эквивалент) для аутентифицированного пользователя - [ ] Можно обновить как минимум `language`; также `nickname`, `timezone`, `phone`, `avatar_url`, `preferences` (и при решении — `social_links`) - [ ] Нельзя сменить email, пароль, role, status через этот эндпоинт - [ ] Без/с невалидным токеном — `401`; невалидное тело — `400` - [ ] Успешный ответ `200` возвращает полный профиль как GET - [ ] OpenAPI/client-swagger и trails обновлены - [ ] Есть покрывающие API-тесты, проходящие в CI - [ ] После успешных тестов обновлена бэкенд-спека (`EventHubBackSpec`) ## Контекст / follow-up - Триггер: EventHubFront — UI i18n уже есть, персист языка на бэкенде ждёт этот API. - Ориентир: `PUT /v1/admin/me` в `admin_handler_me.erl`. - Задачу на Front (форма профиля + sync locale через PUT) **не создавать сейчас**; после готовности бэка — отдельный follow-up. ## Файлы (подсказка) - `src/handlers/handler_user_me.erl` — сейчас только GET - `src/handlers/admin/admin_handler_me.erl` — образец PUT - `src/core/core_user.erl` — `update/2`, `set_field/3` - `src/logic/logic_user.erl` — валидация полей (admin path) - `src/swagger/client-swagger.json` - `EventHubSpec/EventHubBackSpec.md` ## Приоритет P1
cursor-ai added the Task label 2026-07-19 22:02:07 +03:00
cursor-ai self-assigned this 2026-07-19 22:10:04 +03:00
cursor-ai commented 2026-07-19 22:10:04 +03:00
Author
Owner
Copy Link
Copy Source

Беру в работу. Сначала предложу варианты реализации.

Беру в работу. Сначала предложу варианты реализации.
cursor-ai commented 2026-07-19 22:17:11 +03:00
Author
Owner
Copy Link
Copy Source

Реализован вариант 3: PATCH /v1/user/me.

Редактируемые: language (ru|en), nickname, timezone, phone, avatar_url, preferences.
Смена пароля: current_password + password (вместе; неверный текущий → 403).
Неизвестные/запрещённые поля → 400.

Тест: user_test_user_me — passed (OTP 28). Спека EventHubBackSpec обновлена.

Реализован вариант 3: PATCH /v1/user/me. Редактируемые: language (ru|en), nickname, timezone, phone, avatar_url, preferences. Смена пароля: current_password + password (вместе; неверный текущий → 403). Неизвестные/запрещённые поля → 400. Тест: user_test_user_me — passed (OTP 28). Спека EventHubBackSpec обновлена.
cursor-ai commented 2026-07-19 23:27:42 +03:00
Author
Owner
Copy Link
Copy Source

Запушено: Back 0934e2c, Spec e5ab464. PATCH /v1/user/me + unit/API-тесты. Закрываю.

Запушено: Back `0934e2c`, Spec `e5ab464`. PATCH /v1/user/me + unit/API-тесты. Закрываю.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Bug
Дефект или ошибка
 Bug
Дефект или ошибка
 Bug
Дефект или ошибка
 Epic
Крупная инициатива / эпик
 Epic
Крупная инициатива / эпик
 Epic
Крупная инициатива / эпик
 Future
Отложенная работа
 Future
Отложенная работа
 Future
Отложенная работа
 Story
Пользовательская история
 Story
Пользовательская история
 Story
Пользовательская история
 Task
Техническая задача
 Task
Техническая задача
 Task
Техническая задача
No Label Task
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aleksey ci-builder cursor-ai deep-seek
No Assignees cursor-ai
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: EventHub/EventHubBack#48
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?