[wp-trac] Re: [WordPress Trac] #3426: Should include default favicon

[WordPress Trac]

#3426: Should include default favicon
---------------------------------+------------------------------------------
 Reporter:  robertaccettura      |        Owner:  cpoteet 
     Type:  defect (bug)         |       Status:  reopened
 Priority:  normal               |    Milestone:  2.9     
Component:  Performance          |      Version:  2.8     
 Severity:  normal               |   Resolution:          
 Keywords:  favicon needs-patch  |  
---------------------------------+------------------------------------------

Comment(by lloydbudd):

 > I think if the request gets to WordPress then the shared hosts have
 already lost anyway.
 >
 > While we can improve on the performance of serving a 404 page by sending
 out the icon if it gets to WordPress.  I am not sure that is going to make
 it as good for the shared hosts as the files being there.

 I was thinking that it might help a large chunk of scenarios, where they
 would be smart enough to have a caching solution in place, but I'm know
 thinking that solving it there also is not the correct place.


 > Maybe we could add rewrite rules to the htaccess for these specific
 files instead?

 Interesting idea, though haven't htaccess dependent solutions to other
 problems always been frowned upon.

 Looks like the best solution is theme hook based? Insert default favicon
 info if user/theme does not override.

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