diff options
author | 2018-01-11 11:59:46 +0200 | |
---|---|---|
committer | 2018-02-12 17:04:57 +0200 | |
commit | 67559fcdc829341f369f7d2c5b92f1e0dec42f04 (patch) | |
tree | 6527043092abf3a796e9d40a25058b58f7c9f3a0 /moon_wrapper/moon_wrapper/http_server.py | |
parent | 626dc28a44b92a2bfe6473bf0b595dfee71e4d61 (diff) |
adding comments for enahancement
fix handling exception issue
return make response instead of json body
update test case to check on response false instead of status code
Change-Id: Iab8ea20fbe1713e09e79289e8bc73410bf90dc27
Signed-off-by: sgdt6900 <rhanafy.ext@orange.com>
Diffstat (limited to 'moon_wrapper/moon_wrapper/http_server.py')
-rw-r--r-- | moon_wrapper/moon_wrapper/http_server.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/moon_wrapper/moon_wrapper/http_server.py b/moon_wrapper/moon_wrapper/http_server.py index 8027a0d3..f23af182 100644 --- a/moon_wrapper/moon_wrapper/http_server.py +++ b/moon_wrapper/moon_wrapper/http_server.py @@ -3,6 +3,7 @@ # license which can be found in the file 'LICENSE' in this package distribution # or at 'http://www.apache.org/licenses/LICENSE-2.0'. +import flask from flask import Flask, jsonify from flask_restful import Resource, Api import logging @@ -112,13 +113,13 @@ class HTTPServer(Server): def __hook_errors(self): def get_404_json(e): - return jsonify({"result": False, "code": 404, - "description": str(e)}), 404 + return flask.make_response("False") + self.app.register_error_handler(404, get_404_json) def get_400_json(e): - return jsonify({"result": False, "code": 400, - "description": str(e)}), 400 + return flask.make_response("False") + self.app.register_error_handler(400, lambda e: get_400_json) self.app.register_error_handler(403, exceptions.AuthException) |