-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[$1000] Invalid email prefixes aren’t treated as invalid in contact methods page #17839
Comments
Triggered auto assignment to @abekkala ( |
Bug0 Triage Checklist (Main S/O)
|
Job added to Upwork: https://www.upwork.com/jobs/~015a1956a9c964cbc5 |
Current assignee @abekkala is eligible for the External assigner, not assigning anyone new. |
I was able to recreate using an incorrect hyphen placement |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat ( |
Triggered auto assignment to @francoisl ( |
I suggest to put this on hold for #17387 which has possibility to unify all email validations. |
This comment was marked as off-topic.
This comment was marked as off-topic.
ProposalPlease re-state the problem that we are trying to solve in this issue. What is the root cause of that problem?in the src/pages/settings/profile/contacts/newContactMethodPage.js line, no 82 is having inbuilt validation method which looks inappropriate in some cases for the special character at the prefix. What changes do you think we should make in order to solve the problem?we can use regex What alternative solutions did you explore? (Optional)I have explored the third-party packages but it can be inappropriate for some cases. |
Ah good call. I didn't see that one when I looked. Must have used a bad search term. That one was created well before this one so closing. Thanks @parasharrajat |
@parasharrajat @abekkala @MelvinBot i don't think so that regex mentioned in !17387 will include all the test cases for identifying the email totally valid. Here is the proof. I think my regex is more appropriate (passing all the test cases that you have mentioned in the issue except the length but i can add that check as well. ) |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Note :
According to RFC 5322, which defines the standard for email addresses, the local-part (the part before the '@' symbol) can only contain certain characters, including alphanumeric characters, dots, and a limited set of special characters, such as underscores and hyphens. The pound sign (#) is not one of the allowed special characters.
Also, The local-part cannot start or end with a hyphen, so the email address "abc-@mail.com" is considered invalid.
Expected Result:
error handling with improper email address
Actual Result:
Some cases are not checked properly..
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Record_2023-04-22-10-02-37.mp4
Recording.316.mp4
Expensify/Expensify Issue URL:
Issue reported by: @ashimsharma10
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1682146812820119
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: