isFQDN: allow numbers everywhere

[kle-pra]

Hello,

according to FQDN specs ("Only letters, numbers, or dashes can be used."), numbers can be a part of FQDN, not only in TLD (which we can now allow with allow_numeric_tld: true).

Maybe there should be a more general option, alow_numeric.

[sysmat]

Example: abc.efg.g1h. will always fail, current options don't provide this use case, but is valid

validator.isFQDN('abc.efg.g1h.', 
  { allow_trailing_dot: true, allow_underscores: true, allow_numeric_tld: true}); // false but should be true

[pano9000]

@kle-pra @sysmat
it looks like commit 8f04ca9 / PR #1939 may have fixed the issue.

Kindly confirm if that is the case, so that I can close the issue

[kle-pra]

Thank you, It looks OK now when cloning, building & testing file validator.min.js from master branch. The example from @sysmat also works correctly.

Is there also a possibility to npm install master branch directly from github (for example for angular project)?
npm i https://github.com/validatorjs/validator.js.git doesn't seem to work correctly.

[sysmat]

it looks fine. Is there any plan for release or roadmap

[WikiRik]

The idea is that the next release will come out somewhere in the next two weeks

[pano9000]

thanks for your replies. closing this as "fixed" then

[pano9000]

@kle-pra

Is there also a possibility to npm install master branch directly from github (for example for angular project)? npm i https://github.com/validatorjs/validator.js.git doesn't seem to work correctly.

this is down to issue #2123
as some sort of workaround, you could check comment: #2123 (comment)