Skip to content

Fix double email reg issue#215

Merged
Temmmmmo merged 3 commits intomainfrom
Fix-double-email-reg-issue
Oct 16, 2024
Merged

Fix double email reg issue#215
Temmmmmo merged 3 commits intomainfrom
Fix-double-email-reg-issue

Conversation

@DaymasS
Copy link
Copy Markdown
Contributor

@DaymasS DaymasS commented Oct 14, 2024

Изменения

Запрет на привязку почты (Email) к аккаунту у которого уже есть привязанная почта (Email)

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

При попытке привязать почту к аккаунту юзера повторно (post email/registration с токеном сессии юзера ) вернётся код 409 Already exists (Поведение одинаковое, как для почты идентичной привязанной, так и для абсолютно новой почты)

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@DaymasS DaymasS requested a review from Temmmmmo October 14, 2024 06:17
@DaymasS DaymasS self-assigned this Oct 14, 2024
@DaymasS DaymasS linked an issue Oct 14, 2024 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 14, 2024

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
auth_backend
   __main__.py330%1–5
   base.py12467%6–9
   exceptions.py39587%32, 47–49, 54
auth_backend/auth_method
   base.py63494%52, 54, 61, 143
   method_mixins.py32294%27, 50
   oauth.py531670%37, 43, 48–53, 57–68, 84–88
   outer.py1122776%27, 34, 41, 48, 99, 105, 114, 168–173, 186–192, 204–210
   userdata_mixin.py19574%32, 37–40
auth_backend/auth_plugins
   airflow.py331652%26–37, 42–65
   coder.py321650%25–34, 39–52
   email.py2521494%91, 231, 316, 318, 323, 334–340, 387, 428, 479, 489
   github.py1046339%57–116, 127–171, 178, 183, 189–201
   google.py1126443%65–124, 135–163, 170, 176–180, 184–189, 193–202
   keycloak.py1046240%53–115, 122–172, 179, 184, 190–195
   lkmsu.py1379034%59–111, 126–163, 170, 175, 181–195, 199–236, 240–249
   mailu.py341847%25–34, 39–54
   postgres.py432053%29–35, 40–46, 51–63
   telegram.py1116145%56–90, 102–120, 127, 133, 143–163, 167–181
   vk.py1288236%71–123, 134–172, 179, 184, 190–207, 214–240
   yandex.py1278335%58–128, 138–176, 184, 190, 196–222
auth_backend/cli
   group.py13130%1–17
   process.py56560%1–72
   scope.py11110%1–20
   user.py20200%1–40
   user_group.py12120%1–25
auth_backend/kafka
   kafka.py502060%30–33, 42–44, 56–59, 72–82, 85, 93, 103
   kafkameta.py9278%8, 12
auth_backend/models
   base.py83594%79, 100–102, 106
   db.py136398%52, 209, 213
   dynamic_settings.py27774%24–31
auth_backend/routes
   base.py26292%20–21
   exc_handlers.py37586%29, 54, 67, 93, 105
   groups.py84396%85, 135, 137
   scopes.py36392%24, 53–54
   user.py872670%51, 53, 55–64, 70–76, 88–108, 155
   user_session.py84693%76–85, 114, 116, 174, 181
auth_backend/schemas
   models.py98397%148–150
auth_backend/schemas/types
   scopes.py24675%30–32, 37, 40, 42
auth_backend/utils
   security.py43295%36, 61
   smtp.py771877%76–79, 101–120
   string.py7186%12
   user_session_control.py33197%28
TOTAL271188068% 

Summary

Tests Skipped Failures Errors Time
59 6 💤 0 ❌ 0 🔥 10.607s ⏱️

Comment thread tests/test_routes/test_registration.py
@Temmmmmo Temmmmmo merged commit 6cd3a09 into main Oct 16, 2024
@Temmmmmo Temmmmmo deleted the Fix-double-email-reg-issue branch October 16, 2024 18:32
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.

Проблема с привязыванием почты

3 participants