[wp-trac] [WordPress Trac] #31645: Press This: Reject relative URLs when scraping source html

[WordPress Trac]

#31645: Press This: Reject relative URLs when scraping source html
--------------------------+--------------------
 Reporter:  kraftbj       |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  4.2
Component:  Press This    |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:
--------------------------+--------------------

Comment (by kraftbj):

 Tested .5 with the following variations of the canonical URL:
 {{{
 http://example.com/123.html
 https://example.com/123.html
 //example.com/123.html
 /123.html
 123.html
 }}}

 In the first three, the canonical URL is used and the latter two the URL
 is rejected, all as expected.

 We're scraping canonical, shortlink, icon—is there a reason we're okay
 with relative URLs for shortlinks or the icon? Shortlinks appear to be a
 very last resort for a canonical URL. I don't see what we're using the
 icon so far.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31645#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform