[wp-trac] [WordPress Trac] #47398: Remove wp_favicon_request

[WordPress Trac]

#47398: Remove wp_favicon_request
----------------------------+------------------------------
 Reporter:  jonoaldersonwp  |       Owner:  SergeyBiryukov
     Type:  enhancement     |      Status:  reviewing
 Priority:  normal          |   Milestone:  Awaiting Review
Component:  Bootstrap/Load  |     Version:
 Severity:  normal          |  Resolution:
 Keywords:  has-patch seo   |     Focuses:
----------------------------+------------------------------
Changes (by joostdevalk):

 * keywords:  has-patch close seo => has-patch seo


Comment:

 @ocean90 I'm fine debating with you over this, I'm not fine with you just
 adding a `close` keyword: that's just your opinion and you have no more
 and no less authority over this than I do. So adding that like this is
 RUDE and I don't appreciate it.

 By removing this function, we'd allow plugins to hook into the URL with
 the rewrite API, currently they can't because we serve a stupid empty
 favicon. Our current solution is ridiculously poor. Even shipping a
 default WordPress favicon with every install would be better than this, or
 even better: outputting the site icon in the favicon.ico.

 I'm happy to discuss other paths to a better solution of this problem.

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