fixup! NO-SNOW: Add core library import

Author: sfc-gh-turbaszek

src/snowflake/connector/__init__.py

@@ -5,7 +5,7 @@
 
 from functools import wraps
 
-from ._utils import core_loader
+from ._utils import _core_loader
 
 apilevel = "2.0"
 threadsafety = 2
@@ -47,17 +47,17 @@
 from .log_configuration import EasyLoggingConfigPython
 from .version import VERSION
 
-logging.getLogger(__name__).addHandler(NullHandler())
-setup_external_libraries()
-
 # Load the core library - failures are captured in core_loader and don't prevent module loading
 try:
-    core_loader.load()
+    _core_loader.load()
 except Exception:
     # Silently continue if core loading fails - the error is already captured in core_loader
     # This ensures the connector module loads even if the minicore library is unavailable
     pass
 
+logging.getLogger(__name__).addHandler(NullHandler())
+setup_external_libraries()
+
 
 @wraps(SnowflakeConnection.__init__)
 def Connect(**kwargs) -> SnowflakeConnection:

src/snowflake/connector/_utils.py

@@ -4,6 +4,7 @@
 import importlib
 import string
 import sys
+import threading
 from enum import Enum
 from inspect import stack
 from random import choice
@@ -112,7 +113,7 @@ def _get_core_path():
         if sys.platform.startswith("win"):
             lib_name = "libsf_mini_core.dll"
         elif sys.platform.startswith("darwin"):
-            lib_name = "libsf_mini_core.dyldib"
+            lib_name = "libsf_mini_core.dylib"
         else:
             lib_name = "libsf_mini_core.so"
 
@@ -134,20 +135,32 @@ def _load_minicore(path: str) -> ctypes.CDLL:
             core = ctypes.CDLL(str(lib_path))
         return core
 
-    def load(self) -> None:
+    def _load(self) -> None:
         try:
             path = self._get_core_path()
             core = self._load_minicore(path)
             self._register_functions(core)
             self._version = core.sf_core_full_version()
+            self._error = None
         except Exception as err:
             self._error = err
 
+    def load(self):
+        """Spawn a separate thread to load the minicore library (non-blocking)."""
+        self._error = "still-loading"
+        thread = threading.Thread(target=self._load, daemon=True)
+        thread.start()
+
     def get_load_error(self) -> str:
         return str(self._error)
 
-    def get_core_version(self) -> str:
-        return self._version.decode("utf-8") if self._version else self._version
+    def get_core_version(self) -> str | None:
+        if self._version:
+            try:
+                return self._version.decode("utf-8")
+            except Exception:
+                pass
+        return None
 
 
-core_loader = _CoreLoader()
+_core_loader = _CoreLoader()

src/snowflake/connector/auth/_auth.py

@@ -17,7 +17,7 @@
     load_pem_private_key,
 )
 
-from .._utils import core_loader, get_application_path
+from .._utils import _core_loader, get_application_path
 from ..compat import urlencode
 from ..constants import (
     DAY_IN_SECONDS,
@@ -144,8 +144,8 @@ def base_auth_data(
                         platform_detection_timeout_seconds=platform_detection_timeout_seconds,
                         session_manager=session_manager.clone(max_retries=0),
                     ),
-                    "CORE_LOAD_ERROR": core_loader.get_load_error(),
-                    "CORE_VERSION": core_loader.get_core_version(),
+                    "CORE_LOAD_ERROR": _core_loader.get_load_error(),
+                    "CORE_VERSION": _core_loader.get_core_version(),
                 },
             },
         }

src/snowflake/connector/minicore/__init__.py

@@ -0,0 +1,54 @@
+/**
+ * @file sf_mini_core.h
+ * @brief C API for sf_mini_core library
+ *
+ * This header file provides the C API for the sf_mini_core Rust library.
+ * This file is auto-generated by cbindgen. Do not edit manually!
+ */
+
+#ifndef SF_MINI_CORE_H
+#define SF_MINI_CORE_H
+
+/* Warning, this file is autogenerated by cbindgen. Don't modify this manually.
+ */
+
+#include <stdarg.h>
+#include <stdbool.h>
+#include <stdint.h>
+#include <stdlib.h>
+
+#ifdef __cplusplus
+extern "C" {
+#endif  // __cplusplus
+
+/**
+ * Returns the full version string for sf_core.
+ *
+ * This function returns a pointer to a static null-terminated string
+ * containing the version of sf_core.
+ *
+ * @return A pointer to a static string containing the version.
+ *         The caller must NOT free this pointer.
+ *         The returned string is valid for the lifetime of the program.
+ *
+ * @note Thread-safe: Yes
+ * @note This function never returns NULL
+ *
+ * Example usage:
+ * @code
+ *   const char* version = sf_core_full_version();
+ *   printf("Version: %s\n", version);
+ * @endcode
+ *
+ * # Safety
+ *
+ * The returned pointer points to a static string that is valid for the lifetime
+ * of the program. The caller must not free the returned pointer.
+ */
+const char *sf_core_full_version(void);
+
+#ifdef __cplusplus
+}  // extern "C"
+#endif  // __cplusplus
+
+#endif /* SF_MINI_CORE_H */