Bug #92237
closedRemove favicon.ico from URLs that stop rewrite processing
100%
Description
While analyzing my logs for 404 events, i detected a lot of requests to /favicon.ico
. Because I store this file in a different place, I decided to create a static route in the Site Configuration as described in the documentation (https://docs.typo3.org/m/typo3/reference-coreapi/10.4/en-us/ApiOverview/SiteHandling/StaticRoutes.html).
Now a problem arises from the default .htaccess
settings: it stops rewrite processing when this filename is requested, and thus preventing the route to be executed:
# Stop rewrite processing, if we are in the typo3/ directory or any other known directory # NOTE: Add your additional local storages here RewriteRule ^(?:typo3/|fileadmin/|typo3conf/|typo3temp/|uploads/|favicon\.ico) - [L]
I'd like to suggest the removal of the /favicon.ico
from this line:
RewriteRule ^(?:typo3/|fileadmin/|typo3conf/|typo3temp/|uploads/) - [L]
It seems a little odd anyway to have it in here when one reads the comment..
Updated by Gerrit Code Review about 4 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65645
Updated by Gerrit Code Review about 4 years ago
Patch set 1 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65784
Updated by Gerrit Code Review about 4 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65785
Updated by Christian Futterlieb about 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 475a005d409bee70f398838b8ab0a9f0f3161fbd.