mcp-sharepoint-us 2.0.15__tar.gz → 2.0.17__tar.gz

{mcp_sharepoint_us-2.0.15/src/mcp_sharepoint_us.egg-info → mcp_sharepoint_us-2.0.17}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcp-sharepoint-us
-Version: 2.0.15
+Version: 2.0.17
 Summary: SharePoint MCP Server with Microsoft Graph API
 License: MIT
 Project-URL: Homepage, https://github.com/mdev26/mcp-sharepoint-us

{mcp_sharepoint_us-2.0.15 → mcp_sharepoint_us-2.0.17}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "mcp-sharepoint-us"
-version = "2.0.15"
+version = "2.0.17"
 description = "SharePoint MCP Server with Microsoft Graph API"
 readme = "README.md"
 requires-python = ">=3.10"

{mcp_sharepoint_us-2.0.15 → mcp_sharepoint_us-2.0.17}/src/mcp_sharepoint/__init__.py

@@ -43,11 +43,13 @@ def ensure_context(func):
                 client_id = os.getenv("SHP_ID_APP")
                 client_secret = os.getenv("SHP_ID_APP_SECRET")
                 tenant_id = os.getenv("SHP_TENANT_ID")
+                document_library = os.getenv("SHP_DOC_LIBRARY", "Shared Documents")
                 cloud = "government" if ".sharepoint.us" in site_url else "commercial"
 
                 logger.info(f"Site URL: {site_url}")
                 logger.info(f"Tenant ID: {tenant_id}")
                 logger.info(f"Client ID: {client_id}")
+                logger.info(f"Document Library: {document_library}")
                 logger.info(f"Cloud: {cloud}")
 
                 # Create shared authenticator
@@ -70,7 +72,8 @@ def ensure_context(func):
 
                 graph_client = GraphAPIClient(
                     site_url=site_url,
-                    token_callback=get_token
+                    token_callback=get_token,
+                    document_library_name=document_library
                 )
                 logger.info("Graph API client initialized successfully")
 

{mcp_sharepoint_us-2.0.15 → mcp_sharepoint_us-2.0.17}/src/mcp_sharepoint/graph_api.py

@@ -23,16 +23,18 @@ class GraphAPIClient:
     where SharePoint REST API may not support app-only authentication.
     """
 
-    def __init__(self, site_url: str, token_callback):
+    def __init__(self, site_url: str, token_callback, document_library_name: str = "Documents"):
         """
         Initialize Graph API client.
 
         Args:
             site_url: SharePoint site URL (e.g., https://tenant.sharepoint.us/sites/SiteName)
             token_callback: Function that returns access token
+            document_library_name: Name of the document library to use (e.g., "Shared Documents1")
         """
         self.site_url = site_url.rstrip("/")
         self.token_callback = token_callback
+        self.document_library_name = document_library_name
         self._site_id = None
         self._drive_id = None  # Cache drive ID to avoid repeated API calls
 
@@ -105,7 +107,8 @@ class GraphAPIClient:
 
                 # 3. SSL/TLS Test (if HTTPS)
                 if parsed.scheme == "https":
-                    logger.info(f"[TLS] Testing TLS handshake...")
+                    logger.info(f"[TLS] Testing TLS handshake to {hostname}...")
+                    logger.info(f"[TLS] This will attempt to establish encrypted HTTPS connection")
                     context = ssl.create_default_context()
                     try:
                         with context.wrap_socket(sock, server_hostname=hostname) as ssock:
@@ -122,7 +125,22 @@ class GraphAPIClient:
                                 logger.info(f"[TLS] Certificate subject: {subject.get('commonName', 'N/A')}")
                                 logger.info(f"[TLS] Certificate issuer: {dict(x[0] for x in cert['issuer']).get('organizationName', 'N/A')}")
                     except ssl.SSLError as e:
-                        logger.error(f"[TLS] ✗ TLS handshake failed: {e}")
+                        logger.error(f"[TLS] ✗ TLS/SSL handshake failed: {e}")
+                        logger.error(f"[TLS] This could indicate:")
+                        logger.error(f"[TLS]   - Certificate validation failure")
+                        logger.error(f"[TLS]   - TLS version mismatch")
+                        logger.error(f"[TLS]   - Cipher suite incompatibility")
+                        return
+                    except ConnectionResetError as e:
+                        logger.error(f"[TLS] ✗ Connection reset during TLS handshake")
+                        logger.error(f"[TLS] TCP connection was established BUT connection dropped during TLS negotiation")
+                        logger.error(f"[TLS] This indicates:")
+                        logger.error(f"[TLS]   - Firewall is doing deep packet inspection (DPI)")
+                        logger.error(f"[TLS]   - Firewall is blocking TLS connections to {hostname}")
+                        logger.error(f"[TLS]   - SNI (Server Name Indication) filtering is active")
+                        logger.error(f"[TLS]")
+                        logger.error(f"[TLS] SOLUTION: Ask network team to whitelist {hostname} in firewall")
+                        logger.error(f"[TLS] The firewall needs to allow TLS/HTTPS traffic to this endpoint")
                         return
         except socket.timeout:
             logger.error(f"[TCP] ✗ Connection timeout after 10 seconds")
@@ -243,16 +261,32 @@ class GraphAPIClient:
         except requests.exceptions.ConnectionError as e:
             logger.error(f"✗ ConnectionError getting site ID: {e}", exc_info=True)
             logger.error("This indicates the connection was established but then dropped.")
-            logger.error("Running comprehensive diagnostics...")
+            logger.error("Running comprehensive diagnostics to identify the exact failure point...")
+            logger.error("")
 
             # Run diagnostics to help identify the issue
             self._diagnose_connectivity(url)
 
-            logger.error("\nPossible causes:")
-            logger.error("1. Firewall is blocking graph.microsoft.us")
+            logger.error("")
+            logger.error("=" * 70)
+            logger.error("DIAGNOSIS COMPLETE")
+            logger.error("=" * 70)
+            logger.error("")
+            logger.error("Most common causes of 'Connection reset by peer':")
+            logger.error("")
+            logger.error("1. ⚠️  FIREWALL BLOCKING HTTPS/TLS (Most likely based on symptoms)")
+            logger.error("   - TCP connection succeeds")
+            logger.error("   - Connection drops during TLS handshake")
+            logger.error("   - Indicates deep packet inspection (DPI) is active")
+            logger.error("   - Solution: Ask network team to whitelist graph.microsoft.us")
+            logger.error("")
             logger.error("2. Proxy configuration needed")
-            logger.error("3. SSL/TLS version mismatch")
-            logger.error("4. Network instability")
+            logger.error("   - Set HTTP_PROXY and HTTPS_PROXY environment variables")
+            logger.error("")
+            logger.error("3. SSL/TLS version or certificate issue")
+            logger.error("   - Less likely if TCP connects successfully")
+            logger.error("")
+            logger.error("=" * 70)
             raise
 
         except requests.exceptions.Timeout:
@@ -265,7 +299,8 @@ class GraphAPIClient:
 
     def _get_drive_id(self) -> str:
         """
-        Get the default document library drive ID.
+        Get the document library drive ID by name.
+        Fetches all drives and finds the one matching self.document_library_name.
         Caches the result for reuse.
         """
         if self._drive_id:
@@ -273,9 +308,10 @@ class GraphAPIClient:
             return self._drive_id
 
         site_id = self._get_site_id()
-        url = f"{self.graph_endpoint}/sites/{site_id}/drive"
+        url = f"{self.graph_endpoint}/sites/{site_id}/drives"
 
-        logger.info(f"Fetching drive ID from: {url}")
+        logger.info(f"Fetching all drives from: {url}")
+        logger.info(f"Looking for document library named: '{self.document_library_name}'")
 
         try:
             logger.debug(f"Sending GET request to: {url}")
@@ -284,16 +320,37 @@ class GraphAPIClient:
             logger.debug(f"Response received - Status: {response.status_code}")
             self._handle_response(response)
 
-            self._drive_id = response.json()["id"]
-            logger.info(f"✓ Retrieved drive ID: {self._drive_id}")
-            return self._drive_id
+            drives = response.json().get("value", [])
+            logger.info(f"Found {len(drives)} drives in site")
+
+            # Log all available drives for debugging
+            for drive in drives:
+                drive_name = drive.get("name", "Unknown")
+                drive_id = drive.get("id", "Unknown")
+                logger.debug(f"  Drive: '{drive_name}' (ID: {drive_id})")
+
+            # Find the drive matching the configured document library name
+            for drive in drives:
+                if drive.get("name") == self.document_library_name:
+                    self._drive_id = drive["id"]
+                    logger.info(f"✓ Found matching drive '{self.document_library_name}' with ID: {self._drive_id}")
+                    return self._drive_id
+
+            # If no match found, raise an error with helpful message
+            available_drives = [d.get("name", "Unknown") for d in drives]
+            error_msg = (
+                f"Could not find document library named '{self.document_library_name}'. "
+                f"Available drives: {', '.join(available_drives)}"
+            )
+            logger.error(error_msg)
+            raise ValueError(error_msg)
 
         except requests.exceptions.ConnectionError as e:
-            logger.error(f"✗ ConnectionError getting drive ID: {e}", exc_info=True)
+            logger.error(f"✗ ConnectionError getting drives: {e}", exc_info=True)
             raise
 
         except requests.exceptions.RequestException as e:
-            logger.error(f"✗ Network error getting drive ID: {type(e).__name__}: {e}", exc_info=True)
+            logger.error(f"✗ Network error getting drives: {type(e).__name__}: {e}", exc_info=True)
             raise
 
     def list_folders(self, folder_path: str = "") -> List[Dict[str, Any]]:

{mcp_sharepoint_us-2.0.15 → mcp_sharepoint_us-2.0.17/src/mcp_sharepoint_us.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcp-sharepoint-us
-Version: 2.0.15
+Version: 2.0.17
 Summary: SharePoint MCP Server with Microsoft Graph API
 License: MIT
 Project-URL: Homepage, https://github.com/mdev26/mcp-sharepoint-us