Fix parser on get methods

mllaunchpad/api.py

@@ -104,6 +104,7 @@ def _create_request_parser(resource_obj):
             action="append" if is_array else "store",
             choices=p.enum,
             help=str(p.description) + " - Error: {error_msg}",
+            location="args" if resource_obj.method.lower() == "get" else ("json", "values", )
         )
         added_arguments.add(p.name)
 
@@ -176,7 +177,7 @@ def __init__(self, model_api_obj, parser):
 
     def get(self):
         args = self.parser.parse_args(
-            strict=True
+            strict=False
         )  # treats query_params and form_params as interchangeable
         logger.debug("Received GET request with arguments: %s", args)
         return self.model_api.predict_using_model(args)
@@ -190,7 +191,7 @@ def __init__(self, model_api_obj, parser, id_name):
 
     def get(self, some_resource_id):
         args = self.parser.parse_args(
-            strict=True
+            strict=False
         )  # treats query_params and form_params as interchangeable
         args[self.id_name] = some_resource_id
         logger.debug(
@@ -210,7 +211,7 @@ def __init__(self, model_api_obj, query_parser=None, file_parser=None):
 
     def get(self):
         args = self.query_parser.parse_args(
-            strict=True
+            strict=False
         )  # treat query_params and form_params as interchangeable
         logger.debug("Received GET request with arguments: %s", args)
         return self.model_api.predict_using_model(args)

mllaunchpad/cli.py

@@ -126,7 +126,11 @@ def api(settings):
     ModelApi(settings.config, app, debug=True)
     # Flask apps must not be run in debug mode in production, because this allows for arbitrary code execution.
     # We know that and advise the user that this is only for debugging, so this is not a security issue (marked nosec):
-    app.run(debug=True)  # nosec
+    app.run(
+        host=settings.config["api"].get("host"),
+        port=settings.config["api"].get("port"),
+        debug=True
+    )  # nosec
 
 
 @main.command()

tests/test_cli.py

@@ -114,7 +114,8 @@ def test_api(ma, flask, config, runner_cfg_logcfg, caplog):
     )  # Non-production Flask server warning
     flask.assert_called()
     ma.assert_called_with(config, app, debug=True)
-    app.run.assert_called()
+    app.run.assert_called_once()
+    assert list(app.run.call_args[1].keys()) == ["host", "port", "debug"]
 
 
 @mock.patch(