Linkchecker incorrectly reports permanent redirects on internal linkit links

Here is a patch that will invoke the filters prior to linkchecking.

Not sure what happened to the first patch, here is a properly formatted one.

Thanks 🙏

For starters, do you mind fixing the coding standards in your patch? 🤓

Fixed long comment line... thx!

I can sadly confirm this issue. All URLs made by linkit dynamically are unaliased and therefore cause 301 redirect detection.

As linkit is widely used and also the ckeditor integration, I'd say this is a bug.

I created a MR against 2.1.x from the patch in #6 now to give it a try.
@eiriksm what would you say regarding the logics? I'm not experienced enough in this module yet to decide on that.

I'll report if it works at least.

 ------ ---------------------------------------------------------------------- 
  Line   src/LinkExtractorService.php                                          
 ------ ---------------------------------------------------------------------- 
  168    \Drupal calls should be avoided in classes, use dependency injection  
         instead                                                               
 ------ ----------------------------------------------------------------------

Okay I can tell that this at least still fixes the issue with linkit. Link checking now works as expected again and has correct results.

Still some things left to be done:

• Use dependency injection
• Check why tests fail, eventually fix implementation

Static patch for the time being.

Is this not related to the linkchecker settings under "Link Extraction" > Text formats for link extraction (side note, this should probably be labeled "Text filters")? If you check the following filter then the Linkit filter will not run and process your links during extraction.

Linkit URL converter
Updates links inserted by Linkit to point to entity URL aliases.