Removes /logs endpoint

Author: flovilmart

spec/LogsRouter.spec.js

@@ -23,7 +23,7 @@ describe('LogsRouter', () => {
     var router = new LogsRouter();
 
     expect(() => {
-      router.handleGET(request);
+      router.validateRequest(request);
     }).not.toThrow();
     done();
   });
@@ -43,14 +43,14 @@ describe('LogsRouter', () => {
     var router = new LogsRouter();
 
     expect(() => {
-      router.handleGET(request);
+      router.validateRequest(request);
     }).toThrow();
     done();
   });
 
   it('can check invalid master key of request', done => {
     request.get({
-      url: 'http://localhost:8378/1/logs',
+      url: 'http://localhost:8378/1/scriptlog',
       json: true,
       headers: {
         'X-Parse-Application-Id': 'test',

src/Routers/LogsRouter.js

@@ -5,12 +5,16 @@ import * as middleware from "../middlewares";
 export class LogsRouter extends PromiseRouter {
 
   mountRoutes() {
-    this.route('GET','/logs', (req) => {
+    this.route('GET','/scriptlog', middleware.promiseEnforceMasterKeyAccess, this.validateRequest,  (req) => {
       return this.handleGET(req);
     });
-    this.route('GET','/scriptlog', (req) => {
-      return this.handleScriptLog(req);
-    });
+  }
+
+  validateRequest(req) {
+    if (!req.config || !req.config.loggerController) {
+      throw new Parse.Error(Parse.Error.PUSH_MISCONFIGURED,
+        'Logger adapter is not availabe');
+    }
   }
 
   // Returns a promise for a {response} object.
@@ -20,10 +24,15 @@ export class LogsRouter extends PromiseRouter {
   // until (optional) End time for the search. Defaults to current time.
   // order (optional) Direction of results returned, either “asc” or “desc”. Defaults to “desc”.
   // size (optional) Number of rows returned by search. Defaults to 10
+  // n same as size, overrides size if set
   handleGET(req) {
     const from = req.query.from;
     const until = req.query.until;
-    const size = req.query.size;
+    let size = req.query.size;
+    if (req.query.n) {
+      size = req.query.n;      
+    }
+    
     const order = req.query.order
     const level = req.query.level;
     const options = {
@@ -33,27 +42,7 @@ export class LogsRouter extends PromiseRouter {
       order,
       level
     };
-    
-    return req.config.loggerController
-      .getLogs(options)
-      .then(result => ({ response: result }));
-      level,
-    }
-    return this.getLogs(req, options);
-  }
-  
-  handleScriptLog(req) {
-    const size = req.query.n;
-    const level = req.query.level;
-    return this.getLogs(req, { size, level });
-  }
-  
-  getLogs(req, options) {
-    if (!req.config || !req.config.loggerController) {
-      throw new Parse.Error(Parse.Error.PUSH_MISCONFIGURED,
-        'Logger adapter is not availabe');
-    }
-    enforceSecurity(req.auth);
+
     return req.config.loggerController.getLogs(options).then((result) => {
       return Promise.resolve({
         response: result