I am attaching a patch which fixes some headers (newline missing) and modifies the request to a GET as some sites (e.g. arXiv.org) otherwise report errors. Also this patch adds some more information to the report, which could be useful for bigger sites.

CommentFileSizeAuthor
linkchecker.patch2.48 KBWaldemar

Comments

hass’s picture

hass commented
Status: Needs review » Postponed (maintainer needs more info)

Patch will not apply to 2.x and may be obsolete.

Aside - why are you changing HEAD to GET? We don't need the content of the remote site to verify broken link...

hass’s picture

hass commented
Status: Postponed (maintainer needs more info) » Fixed

I have added a logic that detects 405 status code (method not allowed) errors and switch link checking over to GET in this hopefully rare cases. I also found this issue while checking an Amazon product link that gave me a 405 with HEAD.

Showing the maintainer/author of a node would be difficult with the upcoming new UI in 2.x. I think we do not have the space for such a column in many themes. So setting to fixed for now, but I keep this in mind.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.