Skip to content
This repository has been archived by the owner on Mar 7, 2021. It is now read-only.

URNs do not have any generally defined hierarchical components #379

Closed
kareljuricka opened this issue Jul 18, 2017 · 1 comment
Closed

Comments

@kareljuricka
Copy link

Problem:

Problem is in file lib/crawler.js.

On same pages URI.js throws this exceptions where its method absoluteTo is called.

Solution:

I think there should by Try/Catch block around that lines 653+ like it is around that in other cases (line 774 or 850).

var redirectTarget = uri(response.headers.location)
.absoluteTo(robotsTxtUrl)
.normalize();
@fredrikekelund
Copy link
Collaborator

Hi! This was addressed in 9f4e4ec. More details in #363 as well. A patch release with these changes is due soon! Thanks for reporting 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants