mirror of
https://github.com/axllent/mailpit.git
synced 2026-04-26 08:45:54 +03:00
[GH-ISSUE #357] Link Check: localhost links not detected #237
Labels
No labels
awaiting feedback
bug
docker
documentation
enhancement
github_actions
invalid
pull-request
question
stale
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/mailpit#237
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @dregad on GitHub (Sep 4, 2024).
Original GitHub issue: https://github.com/axllent/mailpit/issues/357
Hello @axllent,
I was testing the LinkCheck feature, and noticed that localhost links (which I frequently use while developing) are not detected (http://127.0.0.1 works).
I guess this could be fixed with a small adjustment to the Regex. e.g. appending
|localhostat the end of the 2nd capturing group.github.com/axllent/mailpit@dc1a16ed5c/internal/linkcheck/main.go (L13)PS: many thanks for MailPit, it's awesome ❤️
@dregad commented on GitHub (Sep 4, 2024):
Sorry, forgot to mention this is with v1.18.3. I realize it's somewhat out of date, but I can't upgrade just now, and as far as I can tell the regex is the same in 1.20.3 so I don't think it will make any difference but I'll test again later with the latest version.
@dregad commented on GitHub (Sep 4, 2024):
Same behavior confirmed with v1.20.3.
@axllent commented on GitHub (Sep 5, 2024):
Hi @dregad - I can confirm your issue - the frontend JavaScript was far more relaxed in how it detected a "valid" URL vs: the "backend" for link checking. I have just pushed a fix and released it as part of v1.20.4. Please confirm this new release fixes the issue for you?
@dregad commented on GitHub (Sep 5, 2024):
Thanks for responding so quickly. I think your fix should do the trick, will test and confirm later today!
@dregad commented on GitHub (Sep 5, 2024):
Confirmed working OK in 1.20.4, thank you very much ! 🚀