Skip to content

Исправлено: Для неанонимных отзывов имя пользователя запрашивается че…#168

Merged
petrCher merged 4 commits intomainfrom
fix_anon_comm3
Apr 23, 2026
Merged

Исправлено: Для неанонимных отзывов имя пользователя запрашивается че…#168
petrCher merged 4 commits intomainfrom
fix_anon_comm3

Conversation

@businkv
Copy link
Copy Markdown
Contributor

@businkv businkv commented Apr 23, 2026

…рез UnionAuth._get_userdata. Если в профиле не задано имя, то возвращается ошибка, отзыв не создаётся

Изменения

  1. Исправлена ошибка, из-за которой у неанонимных отзывов поле user_fullname заполнялось как null
    (UnionAuth не отправляет имя пользователя в userdata, а код искал его именно там. В результате значение всегда оставалось null)

  2. Теперь имя пользователя запрашивается через UnionAuth._get_userdata

  3. Если в профиле пользователя не задано имя, то возвращается ошибка, отзыв не создаётся

Детали реализации

  1. В функцию create_comment добавлен параметр request: Request для получения токена из Authorization

  2. Добавлен вызов auth._get_userdata(token, user_id) для получения полных данных пользователя из Auth API

  3. Имя извлекается из поля "Полное имя"

  4. Если имя не найдено, то вызывается ForbiddenAction

…рез UnionAuth._get_userdata. Если в профиле не задано имя, то возвращается ошибка, отзыв не создаётся
@businkv businkv requested a review from petrCher April 23, 2026 17:07
@github-actions
Copy link
Copy Markdown

💩 Code linting failed, use black and isort to fix it.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
rating_api
   __main__.py440%1–7
   exceptions.py43979%35–37, 48–50, 58, 70, 78
rating_api/models
   base.py64198%91
   db.py1645268%86–98, 102–106, 112–126, 132, 167, 171, 175–177, 181–183, 187–189, 199, 213, 222–224, 229, 246–249, 257, 271–283
rating_api/routes
   base.py16194%41
   comment.py1516060%77, 95, 100, 103, 122, 144–145, 167–176, 192, 246–304, 330, 353, 393–404, 431
   exc_handlers.py32584%36, 43, 50, 64, 71
   lecturer.py1096342%61–93, 103–106, 124, 128–136, 180–212, 230, 238, 258–269
rating_api/schemas
   models.py1552485%161, 166–178, 181–185, 188–202
rating_api/utils
   mark.py880%1–20
TOTAL79222771% 

Summary

Tests Skipped Failures Errors Time
88 0 💤 1 ❌ 57 🔥 1m 10s ⏱️

@petrCher petrCher merged commit c85cdf4 into main Apr 23, 2026
2 checks passed
@petrCher petrCher deleted the fix_anon_comm3 branch April 23, 2026 20:04
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.

2 participants