Changeset 13205

Timestamp:

02/18/2010 11:41:50 PM (15 years ago)

Author:

nacin

Message:

Bail early for favicon.ico requests so we don't load WP twice. Props azaozz, sivel. Fixes #3426

Location:

trunk

Files:

• wp-includes/load.php (modified) (1 diff)
• wp-settings.php (modified) (1 diff)

trunk/wp-includes/load.php

@@ -103 +103 @@
     if ( !extension_loaded( 'mysql' ) && !file_exists( WP_CONTENT_DIR . '/db.php' ) )
         die( /*WP_I18N_OLD_MYSQL*/'Your PHP installation appears to be missing the MySQL extension which is required by WordPress.'/*/WP_I18N_OLD_MYSQL*/ );
+}
+
+/**
+ * Don't load all of WordPress when handling a favicon.ico request.
+ * Instead, send the headers for a zero-length favicon and bail.
+ *
+ * @since 3.0.0
+ */
+function wp_favicon_request() {
+        if ( '/favicon.ico' == $_SERVER['REQUEST_URI'] ) {
+                header('Content-Type: image/vnd.microsoft.icon');
+                header('Content-Length: 0');
+                exit;
+        }
 }
 

trunk/wp-settings.php

@@ -43 +43 @@
 // Check for the required PHP version and for the MySQL extension or a database drop-in.
 wp_check_php_mysql_versions();
+
+// Check if we have recieved a request due to missing favicon.ico 
+wp_favicon_request();
 
 // Check if we're in maintenance mode.