Move "get all" keystore configs handler to match convention.

Author: dlongley

lib/http.js

@@ -117,6 +117,30 @@ bedrock.events.on('bedrock-express.configure.routes', app => {
       res.status(201).location(id).json(record.config);
     }));
 
+  // get all keystores by controller query
+  app.get(
+    routes.keystores,
+    cors(),
+    validate({querySchema: getConfigsQuery}),
+    middleware.authorizeZcapInvocation({
+      async getExpectedValues() {
+        const {baseUri, host} = bedrock.config.server;
+        return {
+          host,
+          rootInvocationTarget: `${baseUri}${routes.keystores}`
+        };
+      },
+      async getRootController({req}) {
+        return req.query.controller;
+      }
+    }),
+    asyncHandler(async (req, res) => {
+      const controller = req.query.controller;
+      const options = {projection: {_id: 0, config: 1}, limit: 100};
+      const results = await storage.getAll({controller, req, options});
+      res.json({results});
+    }));
+
   // update keystore config
   app.options(routes.keystore, cors());
   app.post(
@@ -175,30 +199,6 @@ bedrock.events.on('bedrock-express.configure.routes', app => {
       reportOperationUsage({req});
     }));
 
-  // get all keystores with root controller
-  app.get(
-    routes.keystores,
-    cors(),
-    validate({querySchema: getConfigsQuery}),
-    middleware.authorizeZcapInvocation({
-      async getExpectedValues() {
-        const {baseUri, host} = bedrock.config.server;
-        return {
-          host,
-          rootInvocationTarget: `${baseUri}${routes.keystores}`
-        };
-      },
-      async getRootController({req}) {
-        return req.query.controller;
-      }
-    }),
-    asyncHandler(async (req, res) => {
-      const controller = req.query.controller;
-      const options = {projection: {_id: 0, config: 1}, limit: 100};
-      const results = await storage.getAll({controller, req, options});
-      res.json({results});
-    }));
-
   // get a keystore config
   app.get(
     routes.keystore,