Problem/Motivation

Some links, while detected and in the list to check, are never checked. Some of them are invalid domains. Some of them are working links.

Steps to reproduce

Make a basic page with links to http://nonexistentdomainasdf.com and http://eia.gov
Re-scan to add links to list
Run cron
See that links are still listed as "Never" checked

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 BeforePatch.png140.23 KBdivya.sejekan
#5 AfterPatch.png210.96 KBdivya.sejekan
#4 linkchecker-some-links-never-checked-3469263-4.patch1.46 KBschlaukopf

Issue fork linkchecker-3469263

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

ashbeeb created an issue. See original summary.

marc.bau’s picture

marc.bau
Primary language German
Location Germany
commented

What links are never checked? Example?

If non-existend you should get an error. e.g. host name lookup failed.

eahonet’s picture

eahonet commented

Any updates to this one @ashbeeb ? I have a non-existent url (a .com) that continues to show "Last Checked" as "Never".

schlaukopf’s picture

schlaukopf commented
Status: Active » Needs review
StatusFileSize
new linkchecker-some-links-never-checked-3469263-4.patch1.46 KB

I was able to reproduce the same error and found that the issue occurs with domains that don’t exist. Guzzle was unable to resolve the host, which resulted in a ConnectException being thrown instead of a RequestException. This patch addresses the issue by handling TransferException, which will cover both cases.

divya.sejekan’s picture

divya.sejekan commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new AfterPatch.png210.96 KB
new BeforePatch.png140.23 KB

The Patch is applied successfully. Links never checked issue looks resolved , now the links are verified and related error messages . Tried with bunch of different types of URLS.

Steps to reproduce:
1. Make a basic page with links to http://nonexistentdomainasdf.com and http://eia.gov
2. Re-scan to add links to list
3. Run cron
4. See that links are still listed as "Never" checked

RTBC++

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs merge request, +Needs tests

Great work, thanks everyone

If someone could convert this into a merge request, that would be a great start.

It also would be nice to add some tests here. I hope that should be fairly easy, adding to the existing coverage.

Finally. A bit unsure if showing a 502 is more helpful than showing it was never checked. Don't we have other codes for such errors in the codebase already? 🤓

immaculatexavier made their first commit to this issue’s fork.

immaculatexavier opened merge request !124

saidatom’s picture

saidatom
Location Lisbon
commented
Version: 2.0.0 » 2.1.x-dev

saidatom opened merge request !133

saidatom’s picture

saidatom
Location Lisbon
commented
Status: Needs work » Needs review
Issue tags: -Needs merge request, -Needs tests

@eiriksm MR provided with tests. Everything green.