jirimatousek/minify

maxAge is not set to 365 days when query-string ends with number

Closed this issue · 1 comments

Minify commit/version: trunk

What steps will reproduce the problem?
1. append a number to the query string. The config.php says: " * Note: Despite 
this setting, if you include a number at the end of the
 * querystring, maxAge will be set to one year. E.g. /min/f=hello.css&123456"


Expected output:
maxAge to be one year.

Actual output:
maxAge was not changed.


Reason:
regex wrong.

FIX:

This is the culprit:
index.php
if (preg_match('/&\\d/', $_SERVER['QUERY_STRING']) || isset($_GET['v'])) 


my suggested fix:
if (preg_match('/\\d$/', $_SERVER['QUERY_STRING']) || isset($_GET['v'])) {

Original issue reported on code.google.com by leo.saue...@gmail.com on 22 Jul 2014 at 10:30

This must be a problem in your environment. Please post to the Google group.

Original comment by mrclay....@gmail.com on 22 Jul 2014 at 4:27

  • Changed state: Invalid