src/risotto/dispatcher.py

@@ -48,8 +48,8 @@ class CallDispatcher:
                     except AttributeError:
                         err = _(f'function {module_name}.{function_name} return the unknown parameter "{key}" for the uri "{risotto_context.version}.{risotto_context.message}"')
                         raise CallError(err)
-                    except ValueError as err:
+                    except ValueError:
-                        err = _(f'function {module_name}.{function_name} return the invalid parameter "{key}" for the uri "{risotto_context.version}.{risotto_context.message}": {err}')
+                        err = _(f'function {module_name}.{function_name} return the parameter "{key}" with an unvalid value "{value}" for the uri "{risotto_context.version}.{risotto_context.message}"')
                         raise CallError(err)
                     await config.property.read_only()
                     mandatories = await config.value.mandatory()
@@ -89,7 +89,6 @@ class CallDispatcher:
         if hasattr(old_risotto_context, 'connection'):
             # do not start a new database connection
             risotto_context.connection = old_risotto_context.connection
-            risotto_context.log_connection = old_risotto_context.log_connection
             await log.start(risotto_context,
                             kwargs,
                             info_msg,
@@ -109,26 +108,13 @@ class CallDispatcher:
                                         config_arguments,
                                         function_obj,
                                         )
-                await log.success(risotto_context,
+            except CallError as err:
-                                  ret,
-                                  )
-            except Exception as err:
                 await log.failed(risotto_context,
                                  str(err),
                                  )
-                raise CallError(err) from err
+                raise err from err
         else:
-            error = None
-            async with self.pool.acquire() as log_connection:
-                await log_connection.set_type_codec(
-                    'json',
-                    encoder=dumps,
-                    decoder=loads,
-                    schema='pg_catalog'
-                )
-                async with log_connection.transaction():
             try:
-                        risotto_context.log_connection = log_connection
                 async with self.pool.acquire() as connection:
                     await connection.set_type_codec(
                         'json',
@@ -171,22 +157,24 @@ class CallDispatcher:
                                              )
                             raise err from err
             except CallError as err:
-                        error = err
+                raise err from err
             except Exception as err:
                 # if there is a problem with arguments, just send an error and do nothing
                 if get_config()['global']['debug']:
                     print_exc()
+                async with self.pool.acquire() as connection:
+                    await connection.set_type_codec(
+                        'json',
+                        encoder=dumps,
+                        decoder=loads,
+                        schema='pg_catalog'
+                    )
+                    risotto_context.connection = connection
+                    async with connection.transaction():
                         await log.failed(risotto_context,
                                          str(err),
                                          )
-                        error = err
+                raise err from err
-            if error:
-                if not internal:
-                    err = CallError(str(error))
-                    err.context_id = risotto_context.context_id
-                else:
-                    err = error
-                raise err from error
         return ret
 
 

src/risotto/http.py

@@ -101,31 +101,15 @@ async def handle(request):
                             internal=False,
                             **kwargs,
                             )
+    except NotAllowedError as err:
+        raise HTTPNotFound(reason=str(err))
+    except CallError as err:
+        raise HTTPBadRequest(reason=str(err).replace('\n', ' '))
     except Exception as err:
-        context_id = None
-        if isinstance(err, NotAllowedError):
-            error_type = HTTPNotFound
-        elif isinstance(err, CallError):
-            error_type = HTTPBadRequest
-            context_id = err.context_id
-        else:
         if get_config()['global']['debug']:
             print_exc()
-            error_type = HTTPInternalServerError
+        raise HTTPInternalServerError(reason=str(err))
-        response = {'type': 'error',
+    return Response(text=dumps({'response': text}),
-                    'reason': str(err).replace('\n', ' '),
-                    }
-        if context_id is not None:
-            response['context_id'] = context_id
-        err = dumps({'response': response,
-                     'type': 'error',
-                     })
-        raise error_type(text=err,
-                         content_type='application/json',
-                         )
-    return Response(text=dumps({'response': text,
-                                'type': 'success',
-                                }),
                     content_type='application/json',
                     )
 

src/risotto/logger.py

@@ -38,7 +38,7 @@ class Logger:
 
         sql = insert + ') ' + values + ') RETURNING LogId'
         try:
-            log_id = await risotto_context.log_connection.fetchval(sql, *args)
+            log_id = await risotto_context.connection.fetchval(sql, *args)
             if context_id is None and start:
                 risotto_context.context_id = log_id
             if start:
@@ -60,14 +60,14 @@ class Logger:
             sql += ' AND URI = $3'
             args.append(uri)
         ret = []
-        for row in await risotto_context.log_connection.fetch(*args):
+        for row in await risotto_context.connection.fetch(*args):
             d = {}
             for key, value in row.items():
                 if key == 'data':
                     if not value:
                         value = {}
-#                    else:
+                    else:
-#                        value = loads(value)
+                        value = loads(value)
                 elif key in ['start_date', 'stop_date']:
                     value = str(value)
                 d[key] = value
@@ -141,11 +141,7 @@ class Logger:
                     ) -> None:
         paths_msg = self._get_message_paths(risotto_context)
         if get_config()['global']['debug']:
-            if risotto_context.context_id != None:
+            print(_(f'{risotto_context.username}: START:{paths_msg}: {msg}'))
-                context = f'({risotto_context.context_id})'
-            else:
-                context = ''
-            print(_(f'{risotto_context.username}: START{context}:{paths_msg}: {msg}'))
         await self.insert(msg,
                           self._get_last_uri(risotto_context),
                           paths_msg,
@@ -161,7 +157,7 @@ class Logger:
                       ) -> None:
         if get_config()['global']['debug']:
             paths_msg = self._get_message_paths(risotto_context)
-            print(_(f'{risotto_context.username}: SUCCESS({risotto_context.context_id}):{paths_msg}'))
+            print(_(f'{risotto_context.username}: SUCCESS:{paths_msg}({risotto_context.context_id})'))
         sql = """UPDATE RisottoLog
         SET StopDate = $2,
         Level = 'SUCCESS'
@@ -173,7 +169,7 @@ class Logger:
             args.append(dumps(returns))
         sql += """WHERE LogId = $1
         """
-        await risotto_context.log_connection.execute(sql,
+        await risotto_context.connection.execute(sql,
                                                  risotto_context.start_id,
                                                  *args,
                                                  )
@@ -184,18 +180,14 @@ class Logger:
                      ) -> None:
         if get_config()['global']['debug']:
             paths_msg = self._get_message_paths(risotto_context)
-            if risotto_context.context_id != None:
+            print(_(f'{risotto_context.username}: FAILED:{paths_msg}({risotto_context.context_id}): err'))
-                context = f'({risotto_context.context_id})'
-            else:
-                context = ''
-            print(_(f'{risotto_context.username}: FAILED({risotto_context.context_id}):{paths_msg}: err'))
         sql = """UPDATE RisottoLog
         SET StopDate = $2,
         Level = 'FAILED',
         Msg = $3
         WHERE LogId = $1
         """
-        await risotto_context.log_connection.execute(sql,
+        await risotto_context.connection.execute(sql,
                                                  risotto_context.start_id,
                                                  datetime.now(),
                                                  err,

src/risotto/register.py

@@ -297,8 +297,6 @@ class RegisterDispatcher:
                       truncate: bool=False,
                       ) -> None:
         internal_user = get_config()['global']['internal_user']
-        async with self.pool.acquire() as log_connection:
-            async with log_connection.transaction():
         async with self.pool.acquire() as connection:
             await connection.set_type_codec(
                 'json',
@@ -315,7 +313,6 @@ class RegisterDispatcher:
                     risotto_context.username = internal_user
                     risotto_context.paths.append(f'internal.{submodule_name}.on_join')
                     risotto_context.type = None
-                            risotto_context.log_connection = log_connection
                     risotto_context.connection = connection
                     risotto_context.module = submodule_name.split('.', 1)[0]
                     info_msg = _(f'in function risotto_{submodule_name}.on_join')