Kahlid-<br><br>I wrote that page, so I think the answer is a guarded 'yes.'<br><br>We did find some odd behavior when we turned the patch on, which is noted in the Handbook page:<br><br>-- WARNING --<br><br>There is a known issue that occurs if you block .png files with this method. It affects the system_test() function in
system.module, which relies on a phantom .png call to the directory '/system/test/'.<br><br>----<br><br>There may be other bugs caused by our original patch, so I think more testing (especially on 4.7, since the patch was put in place for
4.6) is definitely needed.<br><br>To make debugging easier, I have opened <a href="http://drupal.org/node/76928">http://drupal.org/node/76928</a> and attached the .htaccess file that we currently use.<br><br>I also had to modify function system_view_general() in
system.module to hard-code clean url settings, since the .htaccess rules broke Drupal's clean_url checker.<br><br>- Ken Rickard<br>agentrickard<br><br><br><br>Date: Wed, 2 Aug 2006 23:13:38 -0400<br>From: "Khalid B" <
<a onclick="return top.js.OpenExtLink(window,event,this)" href="mailto:kb@2bits.com">kb@2bits.com</a>><br>Subject: [development] .htaccess patch for 404 of Non-drupal files<br>To: <a onclick="return top.js.OpenExtLink(window,event,this)" href="mailto:development@drupal.org">
development@drupal.org</a><br>Message-ID:<br> <<a onclick="return top.js.OpenExtLink(window,event,this)" href="mailto:4a9fdc630608022013k5b15ae14kdf2fc0195911e510@mail.gmail.com">4a9fdc630608022013k5b15ae14kdf2fc0195911e510@mail.gmail.com
</a>><br>Content-Type: text/plain; charset=ISO-8859-1; format=flowed<br><br>It is worth including the change in this page <a onclick="return top.js.OpenExtLink(window,event,this)" href="http://drupal.org/node/56773" target="_self">
http://drupal.org/node/56773</a><br>to the standard .htaccess we distribute?<br><br>This has two advantages:<br><br>a) Avoid a Drupal bootstrap for every 404 of these files.<br>b) Send a smaller number of bytes when this happen.
<br><br>I believe the majority of sites would not want Drupal to handle 404 for these<br>files.<br><br>I can make the patch if it has a chance of getting through.<br>