Issue 3764 - Got rid of redundant query parameters when redirecting development build updates

modules/downloadserver/files/site.conf

Index: modules/downloadserver/files/site.conf
===================================================================
--- a/modules/downloadserver/files/site.conf
+++ b/modules/downloadserver/files/site.conf
@@ -10,31 +10,31 @@ location /.hg
 # update URL in ABP for Android 1.2. It can be removed once we're
 # not seeing anyone requesting this file anymore.
 # Note that this intentionally promotes devbuilds to release builds.
 # Since newer versions should use the new devbuild and release
 # build update URLs requested by libadblockplus, leaving this
 # redirect around won't really hurt.
 location /devbuilds/adblockplusandroid/updates.xml
 {
-  rewrite ^ https://adblockplus.org/androidupdates.xml?$query_string redirect;
+  rewrite ^ https://adblockplus.org/androidupdates.xml redirect;
 }
 
 location ~ ^(/devbuilds)?/adblockbrowser/updates.xml
 {
   fastcgi_pass unix:/tmp/multiplexer-fastcgi.sock;
   include fastcgi_params;
   fastcgi_cache adblockbrowserupdates;
   fastcgi_cache_valid 200 10m;
   fastcgi_cache_key "$request_uri";
 }
 
 location ~ ^/devbuilds/adblockplus(chrome|chrome-experimental|opera)/updates.xml
 {
-  rewrite ^ https://clients2.google.com/service/update2/crx?$query_string redirect;
+  rewrite ^ https://clients2.google.com/service/update2/crx redirect;
 }
 
 location /devbuilds
 {
   # This is necessary since ABP for Android revisions 323 to 337 crash when
   # detecting an update. Fortunately, we repeated the revision in addonVersion
   # in these versions for some reason, so we can use that pattern to detect
   # them.