[ArtemBratyashin]

Изменения

Добавил проверку email.

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

Функция расширена и в ней добавлено больше условий

Check-List

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

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
auth_backend
__main__.py	3	3	0%	1–4
base.py	12	4	67%	6–9
exceptions.py	45	6	87%	32, 50–52, 57, 78
auth_backend/admin
admin.py	97	37	62%	17, 38, 41, 44–48, 51–55, 58–60, 91, 94, 97–102, 105–113, 116–117, 147, 150, 153–156
auth.py	41	29	29%	14–24, 27–31, 34–35, 39–49
filter.py	17	11	35%	13–24
auth_backend/auth_method
base.py	63	4	94%	51, 53, 60, 142
method_mixins.py	32	2	94%	27, 50
oauth.py	56	17	70%	37, 43, 48–53, 57–68, 94–99
outer.py	114	29	75%	26, 33, 40, 47, 98, 104, 113, 167–172, 185–193, 205–211
userdata_mixin.py	19	5	74%	32, 37–40
auth_backend/auth_plugins
airflow.py	33	16	52%	25–36, 41–64
authentic.py	190	103	46%	52–64, 69–82, 86–101, 105–111, 115–121, 128–130, 140–196, 203–253, 258, 263–264, 274–281, 310–316, 325, 330–340, 345–358
coder.py	32	16	50%	24–33, 38–51
email.py	277	22	92%	36, 47, 49, 51, 54, 58, 62–63, 95, 252, 337, 339, 344, 355–361, 408, 449, 500, 510
github.py	104	63	39%	56–115, 129–173, 183, 188, 194–206
google.py	112	64	43%	64–123, 137–165, 175, 181–185, 189–194, 198–207
keycloak.py	104	62	40%	52–114, 124–174, 184, 189, 195–200
lkmsu.py	164	112	32%	59–112, 130–168, 178–200, 205, 210, 216–226, 230–244, 248–277, 281–290
mailu.py	34	18	47%	24–33, 38–53
postgres.py	43	20	53%	28–34, 39–45, 50–62
telegram.py	103	54	48%	57–84, 99–117, 130, 138, 146–166, 171–185
vk.py	128	82	36%	70–122, 136–174, 184, 189, 195–212, 219–245
yandex.py	127	83	35%	57–127, 140–178, 189, 195, 201–227
auth_backend/cli
group.py	13	13	0%	1–17
process.py	56	56	0%	1–71
scope.py	11	11	0%	1–20
user.py	20	20	0%	1–40
user_group.py	12	12	0%	1–25
auth_backend/kafka
kafka.py	50	20	60%	29–32, 41–43, 55–58, 71–81, 84, 92, 102
kafkameta.py	9	2	78%	8, 12
auth_backend/models
base.py	83	5	94%	79, 100–102, 106
db.py	152	8	95%	46, 55, 59, 120, 214, 223, 227, 240
dynamic_settings.py	27	7	74%	24–31
auth_backend/routes
base.py	37	2	95%	25–26
exc_handlers.py	43	6	86%	31, 56, 69, 95, 117, 125
groups.py	100	3	97%	92, 163, 165
oidc.py	34	1	97%	93
scopes.py	39	3	92%	16, 55–56
user.py	97	27	72%	52, 54, 56–65, 71–77, 89–109, 162, 172
user_session.py	84	6	93%	79–88, 118, 120, 181, 188
auth_backend/schemas
models.py	100	3	97%	150–152
auth_backend/schemas/types
scopes.py	24	6	75%	30–32, 37, 40, 42
auth_backend/utils
jwt.py	57	7	88%	34–41
oidc_token.py	37	8	78%	26, 39–43, 48, 74
security.py	46	2	96%	40, 65
smtp.py	78	19	76%	75–78, 100–120
string.py	7	1	86%	12
user_session_control.py	41	2	95%	32, 40
TOTAL	3336	1112	67%

Summary

Tests	Skipped	Failures	Errors	Time
73	6 💤	0 ❌	0 🔥	13.492s ⏱️

[petrCher]

держи ревью клода, переделай что он говорит, он правильно отревьюил)

[petrCher]

@ArtemBratyashin отредачь эти правки и сделаю мердж

[petrCher]

обьявлен дважды

[petrCher]

объявлен дважды

[petrCher]

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

[petrCher]

в тесте же у домена нет доменной зоны например .com
поэтому 200 при правильном сценарии нельзя получить; сейчас тест пройден, так как отключена проверка в EmailChange
если ее включить тест должен 422 отдавать

[petrCher]

коммент от клода (согласен с ним)

check_email: ручные проверки дублируют validate_email — он сам режет все кейсы из issue (@gmail.com, user@, @, пробелы, кириллицу). Параметр validate не нужен: с False функцию никто не вызывает, и лямбда есть только ради него. Оставь так:

def check_email(v: str) -> str:
    try:
        return validate_email(v, check_deliverability=False, allow_smtputf8=False).normalized
    except EmailNotValidError as exc:
        raise ValueError("Invalid email address") from exc

Подключай без лямбды, через email_validator = field_validator("email")(check_email), в двух схемах:

• EmailRegister — регистрация
• EmailChange — смена почты (новый адрес)

В EmailLogin и RequestResetForgottenPassword валидатор не нужен: там ищем уже существующую почту, и старые адреса должны проходить.

[petrCher]

проблема с тестами:

Тесты падают не из-за PR: SQLAlchemy без версии в requirements подтянулась 2.1 (последняя), а там greenlet больше не ставится по умолчанию — sqladmin падает на импорте.

Временный фикс для этого пр: в requirements.txt заменить SQLAlchemy на SQLAlchemy[asyncio]. psycopg[binary] правильно что добавил, но это только часть проблемы чинит)