You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
In 13.6.0, the string http://foo.com: would cause isURL to fail, which seems correct to me, since the port number is missing. However, in 13.7.0, this string passes validation. This seems wrong.
Examples isURL('http://foo.com:')
Additional context
Validator.js version: 13.7.0
Node.js version: 14.15.0
OS platform: Windows 10, Linux
The text was updated successfully, but these errors were encountered:
After some research, actually this has been introduced in #1751, You can follow the discussion there to understand the reasons of the change (And no it's not a bug)
Thank you
Describe the bug
In 13.6.0, the string
http://foo.com:
would causeisURL
to fail, which seems correct to me, since the port number is missing. However, in 13.7.0, this string passes validation. This seems wrong.Examples
isURL('http://foo.com:')
Additional context
Validator.js version: 13.7.0
Node.js version: 14.15.0
OS platform: Windows 10, Linux
The text was updated successfully, but these errors were encountered: