Added an email request for sending checks to ЮKassa#101
Conversation
refactor: enhance infrastructure stability and fix circular imports
| "msgspec~=0.19.0", | ||
| "cryptography>=46.0.3", | ||
| "qrcode[pil]>=8.2", | ||
| "email-validator>=2.3.0", |
There was a problem hiding this comment.
Стоит ли тянуть зависимость ради одной регулярки?
There was a problem hiding this comment.
Сначала сделал как раз регуляркой. А потом подумал, что у готовой библиотеке возможностей больше, вплоть до проверку на существование почты. Выглядит перспективней. Плюс регулярку универсальную составить довольно сложно. Есть неплохая статья по этому поводу.
Если считаете лишним, уберу. Дайте знать только.
There was a problem hiding this comment.
Статья присланная вами затрагивает очень редкие корнеркейсы описанные в стандарте rfc, 99.9% пользователей используют классические почты username@mail.ru или user.name@mail.ru или username+tag@mail.ru ну их комбинации.
There was a problem hiding this comment.
Убрал библиотеку и заменил на регулярку.
There was a problem hiding this comment.
Зависимость от email-validator уже есть от remnapy - который зависит от pydantic[email]. Его легко найти в uv.lock файле. Так что зависимость все равно используется.
4 participants
No description provided.