mirror of
https://github.com/slurdge/goeland.git
synced 2026-11-11 13:13:58 +00:00
[GH-ISSUE #43] Youtube/Odysee links lost from EEVblog RSS feed #16
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: slurdge/goeland#16
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 @kriswilk on GitHub (Jun 12, 2022).
Original GitHub issue: https://github.com/slurdge/goeland/issues/43
As I was migrating feeds into goeland, I noticed this one seems to lose its video links:
https://www.eevblog.com/feed/
Probably a consequence of how they're referenced in the feed content (the src of an iframe, within a figure). Any way to preserve these?
@slurdge commented on GitHub (Jun 13, 2022):
Yes this is due to the fact that goeland sanitize the feed before applying filters. I could specifically allow some URLs (look at this issue: https://github.com/microcosm-cc/bluemonday/issues/120) but I think it's somewhat not a good solution.
@slurdge commented on GitHub (Jun 13, 2022):
I'm looking to have a special func on iframes what would extract the link on
retrievefilter.@slurdge commented on GitHub (Jun 21, 2022):
I have a current implementation that works but I really don't like it, basically, I do a special processing in order to extract iframe links.
I plan to completely change that with the following:
--unsafe-no-sanitize-filtercommand line option along with a configuration file optionunsafe-no-sanitize-filter.That way, it should be possible to download eevblog RSS, do no sanitation, replace the
@slurdge commented on GitHub (Aug 4, 2022):
I have made the changes in order to not "sanitize" the input and it does work. However it seems that most (all ?) email clients are blocking iframes anyway. @kriswilk Do you have a special email client in mind ?
@slurdge commented on GitHub (Aug 4, 2022):
v0.11.0 should be good for this issue.