[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:  5.3
Component:  Bootstrap/Load  |     Version:
 Severity:  normal          |  Resolution:
 Keywords:  has-patch seo   |     Focuses:
----------------------------+-----------------------------

Comment (by SergeyBiryukov):

 Replying to [comment:13 jonoaldersonwp]:
 > I'm wondering if there might be an advantage to serving the default
 favicon by echoing it out in base64 - that might work around issues or
 rewrite support, installations not in the root, etc?

 * If rewrites/permalinks are disabled, I don't think there's a way to
 catch the request, so serving the icon in base64 probably wouldn't make a
 difference.
 * `do_robots()` only handles the `robots.txt` request if installed at the
 root, see #13115. Unless I'm missing something, the same applies to
 `favicon.ico` as well, it shouldn't be relevant for installations not in
 the root.

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