ifa-nextcloud 0.2.2__tar.gz → 0.2.3__tar.gz

{ifa_nextcloud-0.2.2 → ifa_nextcloud-0.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ifa_nextcloud
-Version: 0.2.2
+Version: 0.2.3
 Summary: Python библиотека для создания ботов в Nextcloud Talk с API, похожим на python-telegram-bot
 Project-URL: Homepage, https://github.com/1FaKe110/nextcloud
 Project-URL: Repository, https://github.com/1FaKe110/nextcloud.git

{ifa_nextcloud-0.2.2 → ifa_nextcloud-0.2.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "ifa_nextcloud"
-version = "0.2.2"
+version = "0.2.3"
 authors = [
     {name = "ifake", email = "your@email.com"},
 ]

{ifa_nextcloud-0.2.2 → ifa_nextcloud-0.2.3}/src/nextcloud/bot/async_bot.py

@@ -405,13 +405,14 @@ class AsyncBot(BotCore):
 
         params = {'lookIntoFuture': 0, 'setReadMarker': 0}
 
+        # params теперь поддерживается
         response = await self.http.post(endpoint, data=data, params=params)
+
         if response.data is not None and response.data != {}:
             logger.success(f"Отправка текстового сообщения в {chat_id}: {text[:50]}...")
             return True
 
         # Fallback: пробуем без параметров
-        logger.trace("Отправка с параметрами не удалась, пробуем без них...")
         response = await self.http.post(endpoint, data=data)
         if response.data is not None and response.data != {}:
             logger.success("Сообщение успешно отправлено (без параметров)")
@@ -809,8 +810,12 @@ class AsyncBot(BotCore):
         Args:
             poll_interval: Интервал опроса для новых комнат
         """
+        # Убеждаемся, что сессия создана
+        await self._ensure_session()
+
         try:
             rooms = await self.get_rooms()
+            logger.info(f"Синхронизация: найдено {len(rooms)} комнат")
 
             current_rooms = set()
             for room in rooms:
@@ -838,6 +843,7 @@ class AsyncBot(BotCore):
 
         except Exception as e:
             logger.error(f"Ошибка при синхронизации комнат: {e}")
+            logger.error(traceback.format_exc())
 
     async def _sync_rooms_loop(self, poll_interval: float = 2, sync_interval: int = 60):
         """
@@ -881,10 +887,25 @@ class AsyncBot(BotCore):
         # Убеждаемся, что сессия создана
         await self._ensure_session()
 
+        # Проверяем аутентификацию
+        status = await self.check_session_status()
+        if not status.get('authenticated'):
+            logger.error(f"Ошибка аутентификации: {status}")
+            logger.error("Проверьте логин и пароль/токен приложения")
+            return
+
+        logger.info(f"Аутентификация успешна как: {status.get('user')}")
+
         # Получаем начальный список комнат
         rooms = await self.get_rooms()
         logger.info(f"Найдено {len(rooms)} доступных комнат")
 
+        if not rooms:
+            logger.warning("Не найдено ни одной комнаты. Проверьте:")
+            logger.warning("1. Что бот добавлен в комнату")
+            logger.warning("2. Правильность токена приложения")
+            logger.warning("3. Права доступа бота")
+
         self.running = True
 
         # Запускаем опрос для каждой комнаты

{ifa_nextcloud-0.2.2 → ifa_nextcloud-0.2.3}/src/nextcloud/bot/core/bot_core.py

@@ -7,6 +7,7 @@ import time
 import os
 import mimetypes
 import threading
+import traceback
 from datetime import datetime
 from typing import Optional, Callable, Dict, Any, List, Union, Tuple
 from loguru import logger
@@ -81,23 +82,45 @@ class BotCore:
         # Способ 1: Проверяем поле messageParameters на наличие файлов
         message_params = msg_data.get('messageParameters', {})
 
-        for key, param in message_params.items():
-            if param.get('type') == 'file':
-                file_info = {
-                    'file_id': param.get('id', ''),
-                    'file_name': param.get('name', 'unknown'),
-                    'file_size': param.get('size', 0),
-                    'mime_type': param.get('mimetype', 'application/octet-stream'),
-                    'file_path': param.get('path', ''),
-                    'download_url': param.get('link', ''),
-                    'direct_link': param.get('directLink', '')
-                }
-                files.append(File(**file_info))
+        # Важно: messageParameters может быть либо словарём, либо списком
+        if isinstance(message_params, dict):
+            for key, param in message_params.items():
+                if param.get('type') == 'file':
+                    file_info = {
+                        'file_id': param.get('id', ''),
+                        'file_name': param.get('name', 'unknown'),
+                        'file_size': param.get('size', 0),
+                        'mime_type': param.get('mimetype', 'application/octet-stream'),
+                        'file_path': param.get('path', ''),
+                        'download_url': param.get('link', ''),
+                        'direct_link': param.get('directLink', '')
+                    }
+                    files.append(File(**file_info))
+        elif isinstance(message_params, list):
+            # Если это список - итерируем напрямую
+            for param in message_params:
+                if isinstance(param, dict) and param.get('type') == 'file':
+                    file_info = {
+                        'file_id': param.get('id', ''),
+                        'file_name': param.get('name', 'unknown'),
+                        'file_size': param.get('size', 0),
+                        'mime_type': param.get('mimetype', 'application/octet-stream'),
+                        'file_path': param.get('path', ''),
+                        'download_url': param.get('link', ''),
+                        'direct_link': param.get('directLink', '')
+                    }
+                    files.append(File(**file_info))
 
         # Способ 2: Проверяем системные сообщения о расшаренных файлах
         system_message = msg_data.get('systemMessage', '')
         if system_message == 'file_shared':
-            file_param = message_params.get('file', {})
+            # message_params может быть словарём или списком
+            if isinstance(message_params, dict):
+                file_param = message_params.get('file', {})
+            else:
+                # Если список, ищем элемент с type='file'
+                file_param = next((p for p in message_params if isinstance(p, dict) and p.get('type') == 'file'), {})
+
             if file_param:
                 file_info = {
                     'file_id': file_param.get('id', ''),
@@ -129,9 +152,19 @@ class BotCore:
 
         # Альтернативный способ через messageParameters
         message_params = msg_data.get('messageParameters', {})
-        if message_params.get('forward'):
-            is_forwarded = True
-            forward_origin = message_params.get('forward', {}).get('name', 'unknown')
+
+        # messageParameters может быть списком или словарём
+        if isinstance(message_params, dict):
+            forward_param = message_params.get('forward')
+            is_forwarded = is_forwarded or (forward_param is not None)
+            if forward_param:
+                forward_origin = forward_param.get('name', 'unknown')
+        elif isinstance(message_params, list):
+            forward_param = next((p for p in message_params if isinstance(p, dict) and p.get('type') == 'forward'),
+                                 None)
+            if forward_param:
+                is_forwarded = True
+                forward_origin = forward_param.get('name', 'unknown')
 
         return is_forwarded, forward_origin
 

{ifa_nextcloud-0.2.2 → ifa_nextcloud-0.2.3}/src/nextcloud/bot/http/async_.py

@@ -72,18 +72,7 @@ class AsyncHTTPClient(BaseHTTPClient):
             retry: bool = True,
             **kwargs
     ) -> HttpResponse:
-        """
-        Внутренний метод выполнения запроса с обработкой 401.
-
-        Args:
-            method: HTTP метод
-            url: Полный URL
-            retry: Флаг повторной попытки
-            **kwargs: Параметры для aiohttp
-
-        Returns:
-            HttpResponse объект
-        """
+        """Внутренний метод выполнения запроса с обработкой 401."""
         await self._ensure_session()
 
         # Добавляем OCS параметр format=json если его нет
@@ -95,7 +84,6 @@ class AsyncHTTPClient(BaseHTTPClient):
 
         # Таймаут по умолчанию
         if 'timeout' not in kwargs:
-            # aiohttp требует объект ClientTimeout
             kwargs['timeout'] = aiohttp.ClientTimeout(total=30)
 
         try:
@@ -105,19 +93,25 @@ class AsyncHTTPClient(BaseHTTPClient):
 
                 # При 401 пробуем переавторизоваться
                 if status_code == 401 and retry:
+                    logger.warning(f"Получен 401 для {url}, пересоздаём сессию...")
                     await self._reinit_session()
                     return await self._make_request(method, url, retry=False, **kwargs)
 
                 # Парсим ответ
                 data = {}
-                if status_code in [200, 201]:
+                if status_code in [200, 201, 207]:  # 207 - Multi-status для PROPFIND
                     try:
-                        json_response = json.loads(raw_text) if raw_text else {}
-                        # Извлекаем ocs.data как в оригинальном боте
-                        data = json_response.get('ocs', {}).get('data', {})
+                        # Для PROPFIND не парсим JSON
+                        if 'xml' in response.headers.get('content-type', ''):
+                            data = {}  # XML обрабатывается отдельно
+                        else:
+                            json_response = json.loads(raw_text) if raw_text else {}
+                            data = json_response.get('ocs', {}).get('data', {})
                     except json.JSONDecodeError:
-                        logger.error(f"Ошибка парсинга JSON: {raw_text[:200]}")
+                        if raw_text and not raw_text.startswith('<?xml'):
+                            logger.error(f"Ошибка парсинга JSON: {raw_text[:200]}")
 
+                logger.debug(f"Request {method} {url} -> {status_code}")
                 return HttpResponse(
                     status_code=status_code,
                     data=data,
@@ -125,6 +119,9 @@ class AsyncHTTPClient(BaseHTTPClient):
                     headers=dict(response.headers)
                 )
 
+        except aiohttp.ClientResponseError as e:
+            logger.error(f"ClientResponseError: {e.status} {e.message}")
+            return HttpResponse(status_code=e.status, data={}, raw_text=str(e))
         except asyncio.TimeoutError:
             logger.error(f"Timeout при запросе к {url}")
             return HttpResponse(status_code=408, data={}, raw_text="Timeout")
@@ -182,10 +179,12 @@ class AsyncHTTPClient(BaseHTTPClient):
             endpoint: str,
             data: Optional[Dict] = None,
             json_data: Optional[Dict] = None,
-            files: Optional[Dict] = None
+            files: Optional[Dict] = None,
+            params: Optional[Dict] = None  # Добавить этот параметр
     ) -> HttpResponse:
         """POST запрос"""
-        return await self.request('POST', endpoint, data=data, json_data=json_data, files=files)
+        return await self.request('POST', endpoint, data=data, json_data=json_data, files=files, params=params)
+
 
     async def put(self, endpoint: str, data: Any, headers: Optional[Dict] = None) -> HttpResponse:
         """

{ifa_nextcloud-0.2.2 → ifa_nextcloud-0.2.3}/src/nextcloud/bot/http/sync.py

@@ -153,10 +153,11 @@ class SyncHTTPClient(BaseHTTPClient):
             endpoint: str,
             data: Optional[Dict] = None,
             json_data: Optional[Dict] = None,
-            files: Optional[Dict] = None
+            files: Optional[Dict] = None,
+            params: Optional[Dict] = None  # Добавить
     ) -> HttpResponse:
         """POST запрос"""
-        return self.request('POST', endpoint, data=data, json_data=json_data, files=files)
+        return self.request('POST', endpoint, data=data, json_data=json_data, files=files, params=params)
 
     def put(self, endpoint: str, data: Any, headers: Optional[Dict] = None) -> HttpResponse:
         """