I'm not sure if this is a bug or a feature, but after installing the module I'm still seeing 404s for HTTP GETs that look like this one (from my raw access log, with dummy IP address data):

1.2.3.4 - - [03/May/2011:16:47:45 -0700] "GET /favicon.ico HTTP/1.1" 404 45 "-" "Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.1; WOW64; Trident/4.0; GTB6.6; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; MSN 9.0;MSN 9.1;MSN 9.6;MSN 10.0; MSNbMSNI; MSNmen-us; MSNcOTH)"

From my cPanel error log:

[Tue May 03 16:47:45 2011] [error] [client 1.2.3.4] File does not exist: /home/name/public_html/favicon.ico

Is this normal?

Comments

sah62’s picture

sah62 commented

Sorry, I should mention that I do in fact have a custom icon specified in my theme, and the icon file exists in my sites/default/files directory.

heatherwoz’s picture

heatherwoz commented

In your .htaccess file you have to comment out the line
RewriteCond %{REQUEST_URI} !=/favicon.ico

It needs to get rewritten as a drupal menu item to work.

sah62’s picture

sah62 commented

That seems to be working - thanks! Is the requirement documented anywhere, though? I don't recall seeing it described anywhere.

robloach’s picture

robloach
he/him
commented
Title: GET /favicon.ico 404 Error » Add documentation about fixing .htaccess
Version: 6.x-1.0 » 7.x-1.x-dev
Component: Code » Documentation
Category: bug » task
Priority: Normal » Major
puddyglum’s picture

puddyglum
Primary language English
Location Elk Grove, CA
commented

This is pretty easy to do.. how can somebody add a documentation page for it?

sah62’s picture

sah62 commented

It's already mentioned in the module's README.txt file. It would probably be helpful to mention it on the project page, too. I don't think a separate documentation page is needed - as you say, it's pretty easy to do.

puddyglum’s picture

puddyglum
Primary language English
Location Elk Grove, CA
commented

Agree. Taken from the Readme:

Drupal 7 Notes

For this module to work properly you need to:

  • enable the core 'Shortcut' module
  • comment out #RewriteCond %{REQUEST_URI} !=/favicon.ico in the Drupal root .htaccess file
johnennew’s picture

johnennew commented
Status: Active » Fixed

Added these notes to the project main page.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.