Hi guys,

As I was reviewing the changes real quick, I noticed a discrepancy which I thought may be a mistake.

The HTML and CSS both do not have the $ at the end whereas they had it before and the other entries have it...

-  <FilesMatch "\.html\.gz$">
+  <FilesMatch "(\.html|\.html\.gz)">
     ForceType text/html
   </FilesMatch>
-  <FilesMatch "\.xml\.gz$">
+  <FilesMatch "(\.xml|\.xml\.gz)$">
     ForceType text/xml
   </FilesMatch>
-  <FilesMatch "\.json\.gz$">
+  <FilesMatch "(\.json|\.json\.gz|\.js|\.js\.gz)$">
     ForceType text/javascript
   </FilesMatch>
-  <FilesMatch "\.css\.gz$">
+  <FilesMatch "(\.css|\.css\.gz)">
     ForceType text/css
   </FilesMatch>
-  <FilesMatch "\.js\.gz$">
-    ForceType text/javascript
-  </FilesMatch>

Thank you.
Alexis Wilke

CommentFileSizeAuthor
#2 boost-636704.patch2.64 KBmikeytown2

Comments

AlexisWilke’s picture

AlexisWilke commented

It is in boosted1.txt too, btw.

mikeytown2’s picture

mikeytown2 commented
Title: Removed $ by mistake in boosted1/2.txt & boost-rules page » Removed $ by mistake in boosted2.txt?
StatusFileSize
new boost-636704.patch2.64 KB

yep this is an opps... root of this has to do with the function I use to generate the rules inside the boost.admin file. attached is a patch that fixes the issue in all 3 files.

It's the HTML and CSS that are effected correct? This is a minor bug, it only effects you if your running a very recent version of apache that doesn't implement T= correctly in the rewrite rules.

mikeytown2’s picture

mikeytown2 commented
Title: Removed $ by mistake in boosted2.txt? » Removed $ by mistake in boosted1/2.txt & boost-rules page
Status: Active » Needs review
mikeytown2’s picture

mikeytown2 commented
Title: Removed $ by mistake in boosted2.txt? » Removed $ by mistake in boosted1/2.txt & boost-rules page
Status: Needs review » Reviewed & tested by the community

committed; leaving as RTBC since 1.16 links to this issue now as a know issue.

AlexisWilke’s picture

AlexisWilke commented

The fix looks good to me.

Yes, only the HTML and CSS were affected.

Thank you.
Alexis Wilke

mikeytown2’s picture

mikeytown2 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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