instantsoft / icms2

Self-hosted Site Management System
https://instantcms.ru
GNU General Public License v2.0
294 stars 120 forks source link

Регистрация пользователей с почтой на кириллическом домене #1317

Open asnenne opened 3 years ago

asnenne commented 3 years ago

Используемая версия InstantCMS: 2

Сейчас в этому случае выдает ошибку, что неверный формат электронной почты:

е44

Однако в движке есть библиотека idna_convert.class, успешно используемая в компоненте "Редиректы". Можно было бы при вводе почты в таком формате преобразовать ее автоматически в punycode.

cmsCore::loadLib('idna_convert.class'); $IDN = new idna_convert(); $IDN->encode($email);

fuzegit commented 3 years ago

На самом деле адрес email в таком случае должен в формате punycode. Сомнительно это имхо.

asnenne commented 3 years ago

Только простые люди-пользователи это понять могут не всегда. Пишут, как знают, в итоге получают ошибку. А текст ошибки не содержит информации, в чем конкретно проблема. А так одной строчкой бы проблема решилась. Мое дело предложить))

fuzegit commented 3 years ago

Строго говоря, я не могу в валидацию email воткнуть конвертацию pubycode, метод валидации должен только проверять, а не изменять. Тут надо наверное у поля делать опцию какую то.

asnenne commented 3 years ago

Перед валидацией можно изменить. Чтобы в валидацию уже в нормальном виде прилетело.