markdown-link-check icon indicating copy to clipboard operation
markdown-link-check copied to clipboard

%23 in URL interpreted as #

Open ndsvw opened this issue 2 years ago • 7 comments

Steps:

  1. Create a file test.md with only this content: https://github.com/atinfo/awesome-test-automation/blob/master/c%23-test-automation.md
  2. Run markdown-link-check test.md

Expected:

  • No failure, because site exists and is online

Actual:

ERROR: 1 dead links found!

  • I think, the %23 is interpreted as the # character and the link checker tries to reach https://github.com/atinfo/awesome-test-automation/blob/master/c, which fails.

ndsvw avatar May 23 '23 17:05 ndsvw

I noticed a similar problem for %2F (i.e. /), giving an error on the link below. Possibly same root cause, that the %2F is not kept but replaced.

https://github.com/eclipse-kuksa/kuksa-python-sdk/pkgs/container/kuksa-python-sdk%2Fkuksa-client

erikbosch avatar Jun 10 '24 13:06 erikbosch

I think the underlying issue here is https://github.com/tcort/link-check/issues/45 - discovered this myself earlier and it puts me off what is otherwise a very useful tool

cc-stjm avatar Oct 11 '24 21:10 cc-stjm

This issue has been marked as stale because it has been open 60 days with no activity. It will be closed in 30 days unless the stale label is removed or someone adds a comment.

github-actions[bot] avatar Apr 10 '25 08:04 github-actions[bot]

comment

ndsvw avatar Apr 11 '25 05:04 ndsvw

This issue has been marked as stale because it has been open 60 days with no activity. It will be closed in 30 days unless the stale label is removed or someone adds a comment.

github-actions[bot] avatar Jun 17 '25 08:06 github-actions[bot]

comment

ndsvw avatar Jun 17 '25 16:06 ndsvw

I was able to reproduce this in the current version 3.13.7, @tcort.

ndsvw avatar Sep 13 '25 10:09 ndsvw