validateURL is not nice function

[YaroslavGaponov]

I mean it possible problems for local hostname like http://myworknotebook etc

[alexlangberg]

Yup. To add to this, it would at least be handy for the error to indicate which of the potentially hundreds of links is invalid. It would be nice to change it to this:

if(!validateURL(entries[i].link))
  throw new Error('invalid entry link');

... to this:

if(!validateURL(entries[i].link))
  throw new Error('invalid entry link: ' + entries[i].link);

[alexlangberg]

It also seems to fail, even on valid urls such as:

https://foo.com/bar/?baz=bax

A suggestion could be to use the validator module:

https://www.npmjs.com/package/validator

Or pick up their version of url validation:

https://github.com/chriso/validator.js/blob/master/validator.js#L200-L260

[goodseller]

+1

[ArtskydJ]

Fixed in 761272c?

[jpmonette]

Removed validation on URL - developer is free to write them valid.