[wp-trac] [WordPress Trac] #11412: Discard requests for favicon.ico

WordPress Trac wp-trac at lists.automattic.com
Sat Dec 12 22:10:38 UTC 2009


#11412: Discard requests for favicon.ico
-------------------------------+--------------------------------------------
 Reporter:  Denis-de-Bernardy  |       Owner:     
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  3.0
Component:  Performance        |     Version:  2.9
 Severity:  normal             |    Keywords:     
-------------------------------+--------------------------------------------

Comment(by Denis-de-Bernardy):

 if a user has has own favicon file, he probable doesn't want the WP
 favicon to overwrite his own. so this ought to be fixed at the php level.
 :-)

 Personally, I'm picturing something like... at the very beginning of the
 main request:

 http://core.trac.wordpress.org/browser/trunk/wp-
 includes/classes.php?rev=11930#L488


 {{{
 if preg_match favicon_request
   then if $file = apply_filters('wp_favicon, false) !== false
    then
     send appropriate headers
     readfile($file);
    endif
     else
    send send 404 headers
   endif
   die
 endif
 }}}

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/11412#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list