Page MenuHomePhabricator

RSS <link>s should be from /, not http://
Closed, ResolvedPublic

Description

Regarding efficient linking,
you've got most of them right, except these few:
$ i=en.wikipedia.org; lynx -source http://$i/wiki/Faraday_cage|perl -nwle 'm@href="[^"]+@&&print $&'|fgrep $i|sort -u
href="http://en.wikipedia.org/apple-touch-icon.png
href="http://en.wikipedia.org/w/index.php?title=Faraday_cage&amp;action=edit
href="http://en.wikipedia.org/w/index.php?title=Special:RecentChanges&amp;feed=atom
href="http://en.wikipedia.org/w/index.php?title=Special:RecentChanges&amp;feed=rss
href="http://en.wikipedia.org/wiki/Faraday_cage
href="http://en.wikipedia.org/wiki/Wikipedia:Text_of_the_GNU_Free_Documentation_License
They could be written just href="/..." etc. like the rest.


Version: 1.15.x
Severity: trivial

Details

Reference
bz17538

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:30 PM
bzimport set Reference to bz17538.
bzimport added a subscriber: Unknown Object (MLST).

Most of these should be a matter of swapping a getFullURL() out for a getLocalURL().

There are places where full URLs are technical required, but <link> isn't one of them. :)

  • This bug has been marked as a duplicate of bug 16357 ***