Issue 3810 - Remove hardcoded default mirror list from Malware Domains List conversion script

sitescripts/subscriptions/bin/updateMalwareDomainsList.py

Index: sitescripts/subscriptions/bin/updateMalwareDomainsList.py
===================================================================
--- a/sitescripts/subscriptions/bin/updateMalwareDomainsList.py
+++ b/sitescripts/subscriptions/bin/updateMalwareDomainsList.py
@@ -11,53 +11,43 @@
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
 #
 # You should have received a copy of the GNU General Public License
 # along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
 
 import os, subprocess, codecs, urllib2, zipfile, tempfile, shutil, sys
-import ConfigParser
 from StringIO import StringIO
 from sitescripts.utils import get_config
 
 
 FILTERLIST_HEADER = '''[Adblock Plus 1.1]
 ! This is a list of malware domains generated from malwaredomains.com data.
 ! Homepage: http://malwaredomains.com/?page_id=2
 ! Last modified: %timestamp%
 ! Expires: 1d
 !'''
 
-DEFAULT_MIRRORS_LIST = [
-    'http://mirror3.malwaredomains.com',
-    'http://mirror1.malwaredomains.com',
-    'http://mirror2.malwaredomains.com'
-]
-
 MALWAREDOMAINS_PATH = '/files/justdomains.zip'
 
 
 def try_mirror(mirror):
   try:
     response = urllib2.urlopen(mirror + MALWAREDOMAINS_PATH)
     return response.read()
   except urllib2.HTTPError:
     return None
 
 
 if __name__ == '__main__':
   config = get_config()
-  repository = config.get('subscriptionDownloads', 'malwaredomains_repository')
-  try:
-    mirrors = config.get('subscriptionDownloads', 'malwaredomains_mirrors')
-    mirrors_list = mirrors.split()
-  except ConfigParser.NoOptionError:
-    mirrors_list = DEFAULT_MIRRORS_LIST
+  section = 'subscriptionDownloads'
+  repository = config.get(section, 'malwaredomains_repository')
+  mirrors_list = config.get(section, 'malwaredomains_mirrors').split()

[Sebastian Noack, 2016/03/16 11:27:34]

I guess we can call this variable simply "mirrors" now.

[Vasily Kuznetsov, 2016/03/16 11:56:59]

And then later we'll have "for mirror in mirrors", yeah, that's nicer indeed.

 
   tempdir = tempfile.mkdtemp(prefix='malwaredomains')
   try:
     subprocess.check_call(['hg', '-q', 'clone',  '-U', repository, tempdir])
     subprocess.check_call(['hg', '-q', 'up', '-R', tempdir, '-r', 'default'])
 
     path = os.path.join(tempdir, 'malwaredomains_full.txt')
     file = codecs.open(path, 'wb', encoding='utf-8')