Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Divide login email errors to Email error and Password error #230

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

physphile
Copy link
Contributor

Изменения

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

Check-List

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

Copy link

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.py561770%38, 44, 49–54, 58–69, 95–100
   outer.py1142975%27, 34, 41, 48, 99, 105, 114, 168–173, 186–194, 206–212
   userdata_mixin.py19574%32, 37–40
auth_backend/auth_plugins
   airflow.py331652%26–37, 42–65
   authentic.py19010346%53–65, 70–83, 87–102, 106–112, 116–122, 129–131, 141–197, 204–254, 259, 264–265, 275–282, 311–317, 326, 331–341, 346–359
   coder.py321650%25–34, 39–52
   email.py2521494%91, 234, 319, 321, 326, 337–343, 390, 431, 482, 492
   github.py1046339%57–116, 130–174, 184, 189, 195–207
   google.py1126443%65–124, 138–166, 176, 182–186, 190–195, 199–208
   keycloak.py1046240%53–115, 125–175, 185, 190, 196–201
   lkmsu.py16411232%60–113, 131–169, 179–201, 206, 211, 217–227, 231–245, 249–278, 282–291
   mailu.py341847%25–34, 39–54
   postgres.py432053%29–35, 40–46, 51–63
   telegram.py1116145%56–90, 105–123, 133, 139, 149–169, 173–187
   vk.py1288236%71–123, 137–175, 185, 190, 196–213, 220–246
   yandex.py1278335%58–128, 141–179, 190, 196, 202–228
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.py137596%52, 56, 210, 214, 226
   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.py942771%54, 56, 58–67, 73–79, 91–111, 162, 179
   user_session.py84693%80–89, 119, 121, 182, 189
auth_backend/schemas
   models.py100397%150–152
auth_backend/schemas/types
   scopes.py24675%30–32, 37, 40, 42
auth_backend/utils
   security.py43295%36, 66
   smtp.py771877%76–79, 101–120
   string.py7186%12
   user_session_control.py35197%29
TOTAL2946101166% 

Summary

Tests Skipped Failures Errors Time
62 6 💤 0 ❌ 0 🔥 11.830s ⏱️

@@ -145,7 +145,7 @@ async def _login(cls, user_inp: EmailLogin, background_tasks: BackgroundTasks) -
.one_or_none()
)
if not query:
raise AuthFailed("Incorrect login or password", "Некорректный логин или пароль")
raise AuthFailed("Email is not registered", "Такая почта еще не зарегистрирована")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

таким образом можно будет понять какие почты зарегестрированы и брутфорсить онли пароль

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