slurdge/goeland

Fork 0

mirror of https://github.com/slurdge/goeland.git synced 2026-11-11 13:13:58 +00:00

[GH-ISSUE #214] No link included despite the includelink filter #40

New issue

Closed

opened 2026-09-29 12:58:58 +00:00 by slurdge · 4 comments

slurdge commented

2026-09-29 12:58:58 +00:00

Owner

Originally created by @Wobak on GitHub (Jul 3, 2024).
Original GitHub issue: https://github.com/slurdge/goeland/issues/214

Describe the bug
No link included despite the includelink filter

To Reproduce
Steps to reproduce the behavior:
I have example emails with the config being correctly sending emails, but no link included

Expected behavior
THe link being included :)

Screenshots
N/A

Desktop (please complete the following information):
Any mail client

Additional context
Config.toml excerpt:

[sources.myblog]
url = "<redacted>"
type = "feed"
filters = ["all", "unseen", "includelink"]
allow-insecure = false

Originally created by @Wobak on GitHub (Jul 3, 2024). Original GitHub issue: https://github.com/slurdge/goeland/issues/214 **Describe the bug** No link included despite the includelink filter **To Reproduce** Steps to reproduce the behavior: I have example emails with the config being correctly sending emails, but no link included **Expected behavior** THe link being included :) **Screenshots** N/A **Desktop (please complete the following information):** Any mail client **Additional context** Config.toml excerpt: ``` [sources.myblog] url = "<redacted>" type = "feed" filters = ["all", "unseen", "includelink"] allow-insecure = false ```

slurdge closed this issue

2026-09-29 12:58:58 +00:00

slurdge commented

2026-09-29 12:58:59 +00:00

Author

Owner

@Wobak commented on GitHub (Jul 3, 2024):

And I can see in the logs that the includelink is indeed seen :

goeland-2024-thibaut  | time="2024-07-03T13:00:00Z" level=debug msg="Executing includelink filter with args: []"
goeland-2024-thibaut  | time="2024-07-03T13:00:00Z" level=info msg="After includelink: 1 feeds"
goeland-2024-thibaut  | time="2024-07-03T13:00:00Z" level=debug msg="After includelink: [{UID:66853888034...

@Wobak commented on GitHub (Jul 3, 2024): And I can see in the logs that the includelink is indeed seen : ``` goeland-2024-thibaut | time="2024-07-03T13:00:00Z" level=debug msg="Executing includelink filter with args: []" goeland-2024-thibaut | time="2024-07-03T13:00:00Z" level=info msg="After includelink: 1 feeds" goeland-2024-thibaut | time="2024-07-03T13:00:00Z" level=debug msg="After includelink: [{UID:66853888034... ```

slurdge commented

2026-09-29 12:58:59 +00:00

Author

Owner

@slurdge commented on GitHub (Jul 9, 2024):

Check if you have:

include-title= true

In you config, or else, you won't generate titles, and therefore no back link.

Alternatively, you can use a custom made template and use the {{.EntryURL}} variable. The template is there:
https://github.com/slurdge/goeland/blob/master/cmd/asset/email.default.html

@slurdge commented on GitHub (Jul 9, 2024): Check if you have: ``` include-title= true ``` In you config, or else, you won't generate titles, and therefore no back link. Alternatively, you can use a custom made template and use the `{{.EntryURL}}` variable. The template is there: https://github.com/slurdge/goeland/blob/master/cmd/asset/email.default.html

slurdge commented

2026-09-29 12:59:00 +00:00

Author

Owner

@Wobak commented on GitHub (Jul 18, 2024):

I added include-link but not include-title. Let me add that and see with the next article, thanks. Is that documented somewhere and I missed it?

@Wobak commented on GitHub (Jul 18, 2024): I added include-link but not include-title. Let me add that and see with the next article, thanks. Is that documented somewhere and I missed it?

slurdge commented

2026-09-29 12:59:00 +00:00

Author

Owner

@Wobak commented on GitHub (Jul 20, 2024):

Works with the include title :)

Thank you for the update, might be worth updating the README to put it in the email section I guess ?

@Wobak commented on GitHub (Jul 20, 2024): Works with the include title :) Thank you for the update, might be worth updating the README to put it in the email section I guess ?

slurdge referenced this issue

2026-09-29 12:59:23 +00:00

[PR #40] [MERGED] chore(deps): bump github.com/tdewolff/minify/v2 from 2.11.7 to 2.11.9 #79