telegram bots: get rid of requests logging via webapi

6 files changed, 9 insertions, 69 deletions

diff --git a/include/py/homekit/api/types/__init__.py b/include/py/homekit/api/types/__init__.py
index 9f27ff6..22ce4e6 100644
--- a/include/py/homekit/api/types/__init__.py
+++ b/include/py/homekit/api/types/__init__.py
@@ -1,5 +1,4 @@
 from .types import (
-    BotType,
     TemperatureSensorDataType,
     TemperatureSensorLocation,
     SoundSensorLocation
diff --git a/include/py/homekit/api/types/types.py b/include/py/homekit/api/types/types.py
index 981e798..294a712 100644
--- a/include/py/homekit/api/types/types.py
+++ b/include/py/homekit/api/types/types.py
@@ -1,17 +1,6 @@
 from enum import Enum, auto
 
 
-class BotType(Enum):
-    INVERTER = auto()
-    PUMP = auto()
-    SENSORS = auto()
-    ADMIN = auto()
-    SOUND = auto()
-    POLARIS_KETTLE = auto()
-    PUMP_MQTT = auto()
-    RELAY_MQTT = auto()
-
-
 class TemperatureSensorLocation(Enum):
     BIG_HOUSE_1 = auto()
     BIG_HOUSE_2 = auto()
diff --git a/include/py/homekit/api/web_api_client.py b/include/py/homekit/api/web_api_client.py
index 15c1915..f9a8963 100644
--- a/include/py/homekit/api/web_api_client.py
+++ b/include/py/homekit/api/web_api_client.py
@@ -57,16 +57,6 @@ class WebApiClient:
     # api methods
     # -----------
 
-    def log_bot_request(self,
-                        bot: BotType,
-                        user_id: int,
-                        message: str):
-        return self._post('log/bot_request/', {
-            'bot': bot.value,
-            'user_id': str(user_id),
-            'message': message
-        })
-
     def log_openwrt(self,
                     lines: List[Tuple[int, str]],
                     access_point: int):
diff --git a/include/py/homekit/database/bots.py b/include/py/homekit/database/bots.py
index cde48b9..fb5f326 100644
--- a/include/py/homekit/database/bots.py
+++ b/include/py/homekit/database/bots.py
@@ -2,7 +2,6 @@ import pytz
 
 from .mysql import mysql_now, MySQLDatabase, datetime_fmt
 from ..api.types import (
-    BotType,
     SoundSensorLocation
 )
 from typing import Optional, List, Tuple
@@ -27,15 +26,6 @@ class OpenwrtLogRecord:
 
 
 class BotsDatabase(MySQLDatabase):
-    def add_request(self,
-                    bot: BotType,
-                    user_id: int,
-                    message: str):
-        with self.cursor() as cursor:
-            cursor.execute("INSERT INTO requests_log (user_id, message, bot, time) VALUES (%s, %s, %s, %s)",
-                           (user_id, message, bot.name.lower(), mysql_now()))
-        self.commit()
-
     def add_openwrt_logs(self,
                          lines: List[Tuple[datetime, str]],
                          access_point: int):
diff --git a/include/py/homekit/telegram/_botutil.py b/include/py/homekit/telegram/_botutil.py
index 111a704..4fbbf28 100644
--- a/include/py/homekit/telegram/_botutil.py
+++ b/include/py/homekit/telegram/_botutil.py
@@ -3,9 +3,6 @@ import traceback
 
 from html import escape
 from telegram import User
-from homekit.api import WebApiClient as APIClient
-from homekit.api.types import BotType
-from homekit.api.errors import ApiResponseError
 
 _logger = logging.getLogger(__name__)
 
@@ -24,20 +21,6 @@ def user_any_name(user: User) -> str:
     return name
 
 
-class ReportingHelper:
-    def __init__(self, client: APIClient, bot_type: BotType):
-        self.client = client
-        self.bot_type = bot_type
-
-    def report(self, message, text: str = None) -> None:
-        if text is None:
-            text = message.text
-        try:
-            self.client.log_bot_request(self.bot_type, message.chat_id, text)
-        except ApiResponseError as error:
-            _logger.exception(error)
-
-
 def exc2text(e: Exception) -> str:
     tb = ''.join(traceback.format_tb(e.__traceback__))
     return f'{e.__class__.__name__}: ' + escape(str(e)) + "\n\n" + escape(tb)
diff --git a/include/py/homekit/telegram/bot.py b/include/py/homekit/telegram/bot.py
index 2e33bea..5ed8b06 100644
--- a/include/py/homekit/telegram/bot.py
+++ b/include/py/homekit/telegram/bot.py
@@ -21,12 +21,10 @@ from telegram.ext.filters import BaseFilter
 from telegram.error import TimedOut
 
 from homekit.config import config
-from homekit.api import WebApiClient
-from homekit.api.types import BotType
 
 from ._botlang import lang, languages
 from ._botdb import BotDatabase
-from ._botutil import ReportingHelper, exc2text, IgnoreMarkup, user_any_name
+from ._botutil import exc2text, IgnoreMarkup
 from ._botcontext import Context
 
 
@@ -39,7 +37,6 @@ _cancel_and_back_filter = filters.Text(lang.all('back') + lang.all('cancel'))
 
 _logger = logging.getLogger(__name__)
 _application: Optional[Application] = None
-_reporting: Optional[ReportingHelper] = None
 _exception_handler: Optional[Coroutine] = None
 _dispatcher = None
 _markup_getter: Optional[callable] = None
@@ -511,22 +508,14 @@ async def _default_any_handler(ctx: Context):
     await ctx.reply(ctx.lang('invalid_command'))
 
 
-def _logging_message_handler(update: Update, context: CallbackContext):
-    if _reporting:
-        _reporting.report(update.message)
-
-
-def _logging_callback_handler(update: Update, context: CallbackContext):
-    if _reporting:
-        _reporting.report(update.callback_query.message, text=update.callback_query.data)
-
-
-def enable_logging(bot_type: BotType):
-    api = WebApiClient(timeout=3)
-    api.enable_async()
-
-    global _reporting
-    _reporting = ReportingHelper(api, bot_type)
+# def _logging_message_handler(update: Update, context: CallbackContext):
+#     if _reporting:
+#         _reporting.report(update.message)
+#
+#
+# def _logging_callback_handler(update: Update, context: CallbackContext):
+#     if _reporting:
+#         _reporting.report(update.callback_query.message, text=update.callback_query.data)
 
 
 def notify_all(text_getter: callable,