NO-SNOW: Refactor _detect_libc to use platform.libc_ver (#2780)

Author: sfc-gh-turbaszek

src/snowflake/connector/_utils.py

@@ -10,7 +10,6 @@
 import time
 from enum import Enum
 from inspect import stack
-from pathlib import Path
 from random import choice
 from threading import Timer
 from uuid import UUID
@@ -188,26 +187,10 @@ def _detect_arch() -> str:
     @staticmethod
     def _detect_libc() -> str:
         """Detect libc type on Linux (glibc vs musl)."""
-        # Check if we're on Alpine/musl
-        if Path("/etc/alpine-release").exists():
-            return "musl"
-
-        # Check for musl by looking at the libc library
-        try:
-            import subprocess
-
-            result = subprocess.run(
-                ["ldd", "--version"],
-                capture_output=True,
-                text=True,
-            )
-            if "musl" in result.stdout.lower() or "musl" in result.stderr.lower():
-                return "musl"
-        except Exception:
-            pass
-
-        # Default to glibc
-        return "glibc"
+        lib, _ = platform.libc_ver()
+        if lib == "glibc":
+            return "glibc"
+        return "musl"
 
     @staticmethod
     def _get_platform_subdir() -> str:

test/unit/test_util.py

@@ -85,17 +85,16 @@ def test_detect_arch(self, machine, expected):
         with mock.patch("platform.machine", return_value=machine):
             assert _CoreLoader._detect_arch() == expected
 
-    def test_detect_libc_alpine(self, tmp_path):
-        """Test _detect_libc returns musl on Alpine Linux."""
-        with mock.patch("pathlib.Path.exists", return_value=True):
+    def test_detect_libc_glibc(self):
+        """Test _detect_libc returns glibc when platform reports glibc."""
+        with mock.patch("platform.libc_ver", return_value=("glibc", "2.31")):
+            assert _CoreLoader._detect_libc() == "glibc"
+
+    def test_detect_libc_musl(self):
+        """Test _detect_libc returns musl when platform does not report glibc."""
+        with mock.patch("platform.libc_ver", return_value=("", "")):
             assert _CoreLoader._detect_libc() == "musl"
 
-    def test_detect_libc_glibc_default(self):
-        """Test _detect_libc returns glibc by default."""
-        with mock.patch("pathlib.Path.exists", return_value=False):
-            with mock.patch("subprocess.run", side_effect=Exception("not found")):
-                assert _CoreLoader._detect_libc() == "glibc"
-
     @pytest.mark.parametrize(
         "os_name,arch,libc,expected_subdir",
         [