Noissue - Do not redirect modern ABB versions to noelemhide filters

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 19 matching lines @@
 {
   set $use_alternative_resource_flags "IE";
 }
 
 if ($arg_application ~ ^(yowser|adblockbrowser)$)
 {
   set $use_alternative_resource_flags "BROWSER";
 }
 
 ### http://hub.eyeo.com/issues/735 Hack for ABB development build that supports elemhide:
-if ($arg_addonVersion ~ ^(2\.)(7|8|9)($|\.).*$)

[mathias, 2017/04/21 14:54:26]

Why is the major version in a capture group? And couldn't the bits after the
minor version (`($|\.).*$`) be replaced by either an optional groupor just a
simple word boundary? So, something like `^2\.(7|8|9)(\..*)?$` or
`^2\.(7|8|9)\b` would probably be a better choice.

+if ($arg_addonVersion ~ ^2\.(7|8|9)(\..*)?$)
 {
   set $use_alternative_resource_flags "${use_alternative_resource_flags}+SUPPORTS_ELEMHIDE";
 }
 
 if ($arg_addonName = "libadblockplus-android")
 {
   set $use_alternative_resource_flags "ZTE";
 }
 
 if ($geoip_country_code = "DE")
@@ skipping 13 matching lines @@
   rewrite ^/easylist\.(.+) /easylist_noelemhide.$1 break;
   rewrite ^/easylistgermany\+easylist\.(.+) /easylistgermany_noelemhide+easylist_noelemhide.$1 break;
   rewrite ^/easylistgermany\.(.+) /easylistgermany_noelemhide.$1 break;
 }
 
 if ($use_alternative_resource_flags = "ZTE")
 {
   rewrite ^/exceptionrules\.(.+) /exceptionrules-minimal.$1 break;
 }