-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Joi.string().invalid (not, disallow) doesn't support regex? #1020
Comments
@pocesar The better choice in my vision is moving your |
I'd say it's duplicate of #867. |
that "reverse" regex from #867 didn't work. that's actually not a "reverse" regex, that's a negative lookhead and it's not the way it's intended to be used. that regex isn't performant and you can get DoS'ed depending on the crafted string, since it's matching and greedy |
Fixed by #1035. |
This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions. |
Context
What are you trying to achieve or the steps to reproduce ?
I'm validating for common unicode full names, with at least one space between name and surname. The regex is actually used to determine if there are any non-name characters in the name.
Which result you had ?
Regex is being ignored on the
not
callWhat did you expect ?
Expected
not
to support regexThe text was updated successfully, but these errors were encountered: