search

magenx / Magento-nginx-config

Default Nginx config for Magento 2
https://www.magenx.com
GNU General Public License v3.0
506 stars 211 forks source link

'location' incompatibilities... #2

Closed Part-A closed 10 years ago

Part-A commented 10 years ago

The two directives in magento.conf:

line 78:   location ~* \.(js|css|png|jpg|jpeg|gif|ico)$ {

and 

line 95:  location ~ .php/ { ## Forward paths like /js/index.php/x.js to relevant handler

are incompatible for 'x.js' with line 78 being evaluated and not 95.

A temporary fix is to remove 'js' from line 78. This will resolve a 404 for javascript used on the admin 'forgot password' screen. This will of course have the effect of removing far future expires for all javascript files.

I will update if I find other affected files elsewhere in Magento.

magenx commented 10 years ago

using new config you have to set proper content-type in /js/index.php