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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ifa_nextcloud
-Version: 0.2.2
+Version: 0.2.4
 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.4}/pyproject.toml

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

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

@@ -393,7 +393,10 @@ class AsyncBot(BotCore):
             return False
 
     async def _send_text_message_async(self, chat_id: str, text: str, reply_to_message_id: int = None) -> bool:
-        """Отправить только текстовое сообщение (асинхронная версия)."""
+        """
+        Отправить только текстовое сообщение (асинхронная версия).
+        Полностью повторяет логику оригинального async_bot.
+        """
         if not text:
             logger.warning("Нет текста для отправки")
             return False
@@ -403,35 +406,27 @@ class AsyncBot(BotCore):
         if reply_to_message_id:
             data['replyTo'] = reply_to_message_id
 
-        params = {'lookIntoFuture': 0, 'setReadMarker': 0}
+        # Отправляем без params, как в оригинале
+        response = await self.http.post(endpoint, data=data)
 
-        response = await self.http.post(endpoint, data=data, params=params)
-        if response.data is not None and response.data != {}:
+        # В оригинале проверяли result is not None and result != {}
+        if response.status_code in [200, 201] and 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("Сообщение успешно отправлено (без параметров)")
-            return True
-
         logger.error("Не удалось отправить текстовое сообщение")
         return False
 
     async def _send_message_with_file_async(
-        self,
-        chat_id: str,
-        text: str,
-        file: Tuple[str, bytes, str],
-        reply_to_message_id: int = None
+            self,
+            chat_id: str,
+            text: str,
+            file: Tuple[str, bytes, str],
+            reply_to_message_id: int = None
     ) -> bool:
         """
         Отправить сообщение с файлом-вложением (асинхронная версия).
-
-        Args:
-            file: Кортеж (filename, content, mime_type)
+        Полностью повторяет логику оригинального async_bot.
         """
         file_name, file_content, mime_type = file
         logger.info(f"📤 Отправка файла {file_name} в комнату {chat_id}")
@@ -482,20 +477,18 @@ class AsyncBot(BotCore):
     async def _get_file_id_async(self, file_name: str) -> str:
         """
         Получить ID загруженного файла через WebDAV PROPFIND (асинхронная версия).
-
-        Returns:
-            ID файла или "unknown"
+        Полностью повторяет логику оригинального async_bot.
         """
         webdav_url = f"/remote.php/dav/files/{self.http.user}/Talk/{file_name}"
 
         propfind_body = '''<?xml version="1.0"?>
-<d:propfind xmlns:d="DAV:" xmlns:oc="http://owncloud.org/ns" xmlns:nc="http://nextcloud.org/ns">
-  <d:prop>
-    <oc:fileid />
-    <oc:size />
-    <d:getlastmodified />
-  </d:prop>
-</d:propfind>'''
+    <d:propfind xmlns:d="DAV:" xmlns:oc="http://owncloud.org/ns" xmlns:nc="http://nextcloud.org/ns">
+      <d:prop>
+        <oc:fileid />
+        <oc:size />
+        <d:getlastmodified />
+      </d:prop>
+    </d:propfind>'''
 
         try:
             response = await self.http.propfind(webdav_url, propfind_body)
@@ -529,9 +522,7 @@ class AsyncBot(BotCore):
     async def _create_public_share_async(self, file_id: str, file_name: str, password: str = None) -> Optional[str]:
         """
         Создать публичную ссылку на файл через Sharing API (асинхронная версия).
-
-        Returns:
-            URL публичной ссылки или None
+        Полностью повторяет логику оригинального async_bot.
         """
         if not file_id or file_id == "unknown":
             return None
@@ -809,8 +800,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 +833,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 +877,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.4}/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
@@ -66,23 +67,22 @@ class BotCore:
     # ========== Вспомогательные методы для работы с API ==========
 
     def _extract_file_from_message(self, msg_data: Dict, chat_id: str) -> List[File]:
-        """
-        Извлечь информацию о файлах из сообщения.
-
-        Args:
-            msg_data: Данные сообщения из API
-            chat_id: ID чата
-
-        Returns:
-            Список File объектов
-        """
+        """Извлечь информацию о файлах из сообщения"""
         files = []
 
         # Способ 1: Проверяем поле messageParameters на наличие файлов
         message_params = msg_data.get('messageParameters', {})
 
-        for key, param in message_params.items():
-            if param.get('type') == 'file':
+        # ВАЖНО: message_params может быть как словарём, так и списком
+        if isinstance(message_params, dict):
+            params_iter = message_params.items()
+        elif isinstance(message_params, list):
+            params_iter = enumerate(message_params)
+        else:
+            params_iter = []
+
+        for key, param in params_iter:
+            if isinstance(param, dict) and param.get('type') == 'file':
                 file_info = {
                     'file_id': param.get('id', ''),
                     'file_name': param.get('name', 'unknown'),
@@ -97,7 +97,13 @@ class BotCore:
         # Способ 2: Проверяем системные сообщения о расшаренных файлах
         system_message = msg_data.get('systemMessage', '')
         if system_message == 'file_shared':
-            file_param = message_params.get('file', {})
+            if isinstance(message_params, dict):
+                file_param = message_params.get('file', {})
+            elif isinstance(message_params, list):
+                file_param = next((p for p in message_params if isinstance(p, dict) and p.get('type') == 'file'), {})
+            else:
+                file_param = {}
+
             if file_param:
                 file_info = {
                     'file_id': file_param.get('id', ''),
@@ -113,25 +119,27 @@ class BotCore:
         return files
 
     def _get_forward_info(self, msg_data: Dict) -> Tuple[bool, Optional[str]]:
-        """
-        Получить информацию о пересылке сообщения.
-
-        Returns:
-            (is_forwarded, forward_origin)
-        """
+        """Получить информацию о пересылке сообщения"""
         is_forwarded = False
         forward_origin = None
 
-        # Проверяем наличие информации о пересылке
         if msg_data.get('isForwarded'):
             is_forwarded = True
             forward_origin = msg_data.get('forwardedFrom', {}).get('actorDisplayName', 'unknown')
 
-        # Альтернативный способ через messageParameters
         message_params = msg_data.get('messageParameters', {})
-        if message_params.get('forward'):
-            is_forwarded = True
-            forward_origin = message_params.get('forward', {}).get('name', 'unknown')
+
+        if isinstance(message_params, dict):
+            forward_param = message_params.get('forward')
+            if forward_param:
+                is_forwarded = True
+                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.4}/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.trace(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.4}/src/nextcloud/bot/http/sync.py

@@ -112,6 +112,7 @@ class SyncHTTPClient(BaseHTTPClient):
             logger.error(f"Ошибка запроса: {e}")
             return HttpResponse(status_code=0, data={}, raw_text=str(e))
 
+
     def request(
             self,
             method: str,
@@ -153,10 +154,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:
         """