From 57a6fa48a7c5c581db50f9590b9ad6c4be43ac2d Mon Sep 17 00:00:00 2001 From: sgdt6900 Date: Thu, 11 Jan 2018 11:57:43 +0200 Subject: adding some comments to enahnce removing comments and add todo point Signed-off-by: sgdt6900 Change-Id: I38f21bd7557fb758b50d561a06efe8ea09ca07fc --- docs/TODO.md | 2 ++ moon_interface/moon_interface/http_server.py | 6 ------ moon_interface/moon_interface/server.py | 8 ++------ 3 files changed, 4 insertions(+), 12 deletions(-) diff --git a/docs/TODO.md b/docs/TODO.md index afdadf3c..caca158a 100644 --- a/docs/TODO.md +++ b/docs/TODO.md @@ -21,6 +21,8 @@ Bugs to fix: - When adding user or VM in GUI, there is a bug in the backend (manager ?) - GUI: in the "Projects" tab, move the "Map" link in the "Action" button - GUI: move tabs in this order : "Models, Policy, PDP, Projects" +- Fixing Error Handling at module "Interface" & "Wrapper" according to this link 'http://flask-restful.readthedocs.io/en/0.3.5/extending.html#custom-error-handlers' +- in case an error occurred, it would be better throwing an exception rather than sending result with error code (as the exception was already thrown in some cases ) [ Interface, Wrapper] Other actions: diff --git a/moon_interface/moon_interface/http_server.py b/moon_interface/moon_interface/http_server.py index 82ee5d91..a2e25377 100644 --- a/moon_interface/moon_interface/http_server.py +++ b/moon_interface/moon_interface/http_server.py @@ -114,12 +114,6 @@ class HTTPServer(Server): def get_400_json(e): return jsonify({"result": False, "code": 400, "description": str(e)}), 400 - ''' - [Note] i have tried to simulate authz post request (authz_Requests/run) to return 500 response code - and an AuthzException thrown from their [Line 63] and catched here , then the server here return response - with 403 code [Forbidden] , is it correct if so why sometime at authz [from Line 137] return a response - with error code , i think we can do it in the same way as the one mentioned? - ''' self.app.register_error_handler(400, lambda e: get_400_json) self.app.register_error_handler(403, exceptions.AuthException) diff --git a/moon_interface/moon_interface/server.py b/moon_interface/moon_interface/server.py index d38dae28..29403a6b 100644 --- a/moon_interface/moon_interface/server.py +++ b/moon_interface/moon_interface/server.py @@ -16,9 +16,7 @@ def create_server(): conf = configuration.get_configuration("components/pipeline").get( "components/pipeline", {}).get("interface", {}) - ''' - [Note] i think pipeline should be changed to interface - ''' + hostname = conf.get("hostname", "pipeline") port = conf.get("port", 80) bind = conf.get("bind", "127.0.0.1") @@ -26,9 +24,7 @@ def create_server(): hostname = "interface" bind = "127.0.0.1" port = 80 - ''' - [Note] i think pipeline should be changed to interface - ''' + configuration.add_component(uuid="pipeline", name=hostname, port=port, -- cgit 1.2.3-korg