Use luci-config for infrequently changing settings, part 2.

appengine/auth_service/handlers_frontend.py

 # Copyright 2014 The Swarming Authors. All rights reserved.
 # Use of this source code is governed by the Apache v2.0 license that can be
 # found in the LICENSE file.
 
 """This module defines Auth Server frontend url handlers."""
 
 import os
 
 import webapp2
 
@@ skipping 76 matching lines @@
   expiration_sec = 24 * 3600
   secret_key = auth.SecretKey('link_ticket_token', scope='local')
   version = 1
 
 
 class ImporterConfigHandler(auth.ApiHandler):
   """Reads and sets configuration of the group importer."""
 
   @auth.require(auth.is_admin)
   def get(self):
-    self.send_response({'config': importer.read_config()})
+    self.send_response({'config': importer.read_config_text()})
 
   @auth.require(auth.is_admin)
   def post(self):
     if config.is_remote_configured():
       self.abort_with_error(409, text='The configuration is managed elsewhere')
-    conf = self.parse_body().get('config')
-    if not importer.is_valid_config(conf):
-      self.abort_with_error(400, text='Invalid config format.')
-    importer.write_config(conf)
+    try:
+      importer.write_config_text(self.parse_body().get('config'))
+    except ValueError as ex:
+      self.abort_with_error(400, text=str(ex))
     self.send_response({'ok': True})
 
 
 class ServiceListingHandler(auth.ApiHandler):
   """Lists registered replicas with their state."""
 
   @auth.require(auth.is_admin)
   def get(self):
     services = sorted(
         replication.AuthReplicaState.query(
@@ skipping 158 matching lines @@
       env_callback=get_additional_ui_environment)
   template.bootstrap({'auth_service': TEMPLATES_DIR})
 
   # Add a fake admin for local dev server.
   if utils.is_local_dev_server():
     auth.bootstrap_group(
         auth.ADMIN_GROUP,
         [auth.Identity(auth.IDENTITY_USER, 'test@example.com')],
         'Users that can manage groups')
   return webapp2.WSGIApplication(get_routes(), debug=debug)