Resolve relative href during discovery #417
Comments
IgorA100
added a commit
to IgorA100/feed-io
that referenced
this issue
Oct 14, 2023
Feed discovery with relative URLs in links Example: https://k47.cz/ Solution to the problem: alexdebril#417
This was referenced Oct 14, 2023
Merged
alexdebril
pushed a commit
that referenced
this issue
Oct 16, 2023
* Fix: Relative URLs in links Feed discovery with relative URLs in links Example: https://k47.cz/ Solution to the problem: #417 * Update Feed.php (Fix relative URLs in links) We will replace the links (add the host if it doesn’t exist) in the content as well * Update Node.php (Fix relative URLs in links) * Update Reader.php (Fix relative URLs in links) Remember the URL for further processing * Update Link.php (Fix relative URLs in links) * Update XmlParser.php (Fix relative URLs in links) * Update Explorer.php * Update Node.php Delete tabs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, if a page contains a feed link with, e.g.,
href="/rss.xml"orhref="rss.xml", feed discovery will return the relative URL as is, instead of resolving it to an absolute URL.I don't really have any experience with PHP, but if you could give some pointers on how to implement it. There was an existing PR in nextcloud/news#1631, but that seems to depend on
pear/net_url2, and I can understand if you do not want to add another dependency just for this.The text was updated successfully, but these errors were encountered: