#12255 - Serve compliance lists to extensions in China

modules/filterserver/files/site.conf

 root /var/www/easylist;
 keepalive_timeout 0;
 charset utf-8;
 gzip_static on;
 default_type text/plain;
 
 # http://hub.eyeo.com/issues/262
 location /filters/easylist-min.txt
 {
   proxy_cache filters;
@@ skipping 42 matching lines @@
 }
 
 # http://hub.eyeo.com/issues/1374
 # http://hub.eyeo.com/issues/10493
 # http://hub.eyeo.com/issues/10517
 if ($arg_application ~ (com\.microsoft\.emmx|com\.ume\.browser\.global|org\.chromium\.chrome))
 {
     set $use_alternative_resource_flags "${use_alternative_resource_flags}+MIN";
 }
 
+# http://hub.eyeo.com/issues/12255
+if ($arg_addonName ~ ^(adblockplus[chrome|firefox|edge|opera])$)
+{
+  set $use_alternative_resource_flags "ABP_WEB_EXTENSION";
+}
+
 ### http://hub.eyeo.com/issues/3924 for Chinese SBrowser video exceptions (1/4)
 if ($arg_application = "sbrowser")
 {
     set $use_alternative_resource_flags "SBROWSER";
 }
 
 ### http://hub.eyeo.com/issues/3924 for Chinese SBrowser video exceptions (2/4)
 if ($arg_addonName = "adblockplussbrowser")
 {
   set $use_alternative_resource_flags "${use_alternative_resource_flags}+ABPSBROWSER";
@@ skipping 38 matching lines @@
   rewrite ^/easylistlithuania\+easylist\.(.+) /easylistlithuania+easylist-minified.$1 break;
   rewrite ^/easylistpolish\+easylist\.(.+) /easylistpolish+easylist-minified.$1 break;
   rewrite ^/easylistspanish\+easylist\.(.+) /easylistspanish+easylist-minified.$1 break;
   rewrite ^/latvianlist\+easylist\.(.+) /latvianlist+easylist-minified.$1 break;
   rewrite ^/liste_fr\+easylist\.(.+) /liste_fr+easylist-minified.$1 break;
   rewrite ^/liste_ar\+liste_fr\+easylist\.(.+) /liste_ar+liste_fr+easylist-minified.$1 break;
   rewrite ^/rolist\+easylist\.(.+) /rolist+easylist-minified.$1 break;
   rewrite ^/ruadlist\+easylist\.(.+) /ruadlist+easylist-minified.$1 break;
 }
 
+# http://hub.eyeo.com/issues/12255
 ### http://hub.eyeo.com/issues/3924 for Chinese SBrowser video exceptions (4/4)
-if ($use_alternative_resource_flags = "SBROWSER+ABPSBROWSER+CN")
+if ($use_alternative_resource_flags ~ ((SBROWSER\+ABPSBROWSER|ABP_WEB_EXTENSION)\+CN))

[mathias, 2018/07/16 15:56:23]

I know we don't exactly need it here, but still, the regular expression should
feature `^` and `$` to make sure it applies to the entire flags string. Or am I
mistaken here and `if`/`~` conditions always imply the entire string being
matched? Don't think so though.

 {
   rewrite ^/easylistchina\+easylist\.(.+) /easylistchina+easylistchina_compliance+easylist.$1 break;
 }