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

이메일 형식 검사 규칙이 다른 문제 #2313

Closed
dewekk opened this issue Apr 16, 2024 · 2 comments
Closed

이메일 형식 검사 규칙이 다른 문제 #2313

dewekk opened this issue Apr 16, 2024 · 2 comments
Labels
bug 버그로 확인된 사항 patch available 패치 있음 표시 (bug와 함께 사용)

Comments

@dewekk
Copy link
Contributor

dewekk commented Apr 16, 2024

회원 정보 항목 검사 함수 _checkSignUpFields 에서 Mail::isVaildMailAddress를 통해 이메일 형식을 검사하고 있습니다.

if ($formInfo->name === 'email_address' && $args->{$formInfo->name} && !Mail::isVaildMailAddress($args->{$formInfo->name}))

다만, 예) test@test admin@localhost 같은 항목은 통과하지만, DB column filter에서 통과하지 못해 아래와 같은 오류 메시지가 발생합니다.
Variable email_address for column email_address must contain a valid e-mail address

if (!preg_match('/^[\w-]+((?:\.|\+|\~)[\w-]+)*@[\w-]+(\.[\w-]+)+$/', $value))

어느 한쪽으로 형식이 통일되면 좋을 것 같습니다.

@dewekk
Copy link
Contributor Author

dewekk commented Apr 16, 2024

위와 같은 일반적이지 않은 형식을 사용하고자 하는 것은 아닙니다.
회원 정보 항목에서도 DB column filter와 동일하게 하여, 유효하지 않은 이메일이라는 한글 메시지가 표시되었으면 합니다.

@kijin
Copy link
Member

kijin commented Apr 16, 2024

네, 적용되는 규칙에 일관성이 없는 것은 버그입니다.

@kijin kijin added the bug 버그로 확인된 사항 label Apr 16, 2024
@kijin kijin added the patch available 패치 있음 표시 (bug와 함께 사용) label Apr 17, 2024
@kijin kijin closed this as completed in 848a45c May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 버그로 확인된 사항 patch available 패치 있음 표시 (bug와 함께 사용)
Projects
None yet
Development

No branches or pull requests

2 participants