FEAT: Build process for Mac Support (#68)

This pull request introduces improvements to the Python library
detection process, updates dependency handling for macOS dynamic
libraries, enhances security in connection string sanitization, and
refactors the `setup.py` file to use a different module for wheel
building.

### Python library detection improvements:
*
[`mssql_python/pybind/CMakeLists.txt`](diffhunk://#diff-dbb5892fbbb28149d1639664797cf3adb48ced28ec11aba95f2e2b338ca46badR158-R180):
Added logic to handle cases where `PYTHON_LIB_DIR` is not set or is
empty on Windows, using Python's `sysconfig` to locate the library
directory. Additionally, support for a custom environment variable
`CUSTOM_PYTHON_LIB_DIR` was added to override the library directory.

### Dependency handling updates for macOS:
*
[`mssql_python/pybind/configure_dylibs.sh`](diffhunk://#diff-4cf7dbf7135a58c02b5bf5dc2794aa826d898aeded724d3716f909dd0cede5ffL69-R98):
Updated `install_name_tool` commands to use `@loader_path` for dynamic
library paths, ensuring better portability. Added commands to set
library IDs explicitly using `@loader_path`.

### Security enhancement in connection string sanitization:
*
[`mssql_python/pybind/ddbc_bindings_mac.cpp`](diffhunk://#diff-5191ba8555f89ff1462b69ab9949c75d79f982904ad2c6bb5c4c9cf90ea0ea34L1086-R1113):
Improved the `SanitizeConnectionString` function to perform
case-insensitive removal of sensitive parameters (`UID` and `Pwd`) by
converting the string to lowercase for comparison.

### Refactor in wheel building:
*
[`setup.py`](diffhunk://#diff-60f61ab7a8d1910d86d9fda2261620314edcae5894d5aaa236b821c7256badd7L5-R5):
Changed the import from `setuptools.command.bdist_wheel` to
`wheel.bdist_wheel` for building platform-specific wheels.

Author: bewithgaurav

mssql_python/libs/macos/lib/libltdl.7.dylib

@@ -102,6 +102,7 @@ else()
     message(STATUS "Auto-detected pybind11 include directory: ${PYBIND11_INCLUDE_DIR}")
 endif()
 
+
 # Find Python Library for linking
 execute_process(
     COMMAND python -c "import sysconfig; print(sysconfig.get_config_var('LIBDIR'))"
@@ -154,8 +155,29 @@ if(APPLE AND (NOT PYTHON_LIB_DIR OR "${PYTHON_LIB_DIR}" STREQUAL ""))
         message(STATUS "Python library not found in standard locations, will try using framework")
         # When using a framework, we don't specify the library directly
     endif()
+elseif(WIN32 AND (NOT PYTHON_LIB_DIR OR "${PYTHON_LIB_DIR}" STREQUAL "" OR "${PYTHON_LIB_DIR}" STREQUAL "None"))
+    # On Windows, if PYTHON_LIB_DIR is not set, we will try to find it using sysconfig
+    message(STATUS "PYTHON_LIB_DIR is None or empty for Windows, trying to find it using sysconfig")
+    # Use Python Executable to find the library directory
+    execute_process(
+        COMMAND python -c "import sys, os; print(os.path.dirname(sys.executable))"
+        OUTPUT_VARIABLE PYTHON_EXEC_DIR
+        OUTPUT_STRIP_TRAILING_WHITESPACE
+    )
+    message(STATUS "Python executable directory: ${PYTHON_EXEC_DIR}")
+    # Set the library directory based on the executable directory
+    set(PYTHON_LIB_DIR "${PYTHON_EXEC_DIR}/libs")
+    message(STATUS "Using Python library directory: ${PYTHON_LIB_DIR}")
 endif()
 
+if(DEFINED ENV{CUSTOM_PYTHON_LIB_DIR} AND NOT "$ENV{CUSTOM_PYTHON_LIB_DIR}" STREQUAL "" AND NOT "$ENV{CUSTOM_PYTHON_LIB_DIR}" STREQUAL "None")
+    message(STATUS "CUSTOM_PYTHON_LIB_DIR is set, using it")
+    set(PYTHON_LIB_DIR $ENV{CUSTOM_PYTHON_LIB_DIR})
+    message(STATUS "Using custom Python library directory: ${PYTHON_LIB_DIR}")
+endif()
+
+message(STATUS "Final Python library directory: ${PYTHON_LIB_DIR}")
+
 # Determine which source file to use based on platform
 if(APPLE)
     set(DDBC_SOURCE "ddbc_bindings_mac.cpp")

mssql_python/libs/macos/lib/libmsodbcsql.18.dylib

@@ -66,22 +66,36 @@ done <<< "$OTOOL_LIST"
 if [ -n "$OLD_LIBODBCINST_PATH" ]; then
   echo "Fixing libmsodbcsql.18.dylib dependency on libodbcinst.2.dylib..."
   echo "  Changing: $OLD_LIBODBCINST_PATH"
-  echo "  To: $LIBODBCINST_PATH"
-  install_name_tool -change "$OLD_LIBODBCINST_PATH" "$LIBODBCINST_PATH" "$LIBMSODBCSQL_PATH"
+  echo "  To: @loader_path/libodbcinst.2.dylib"
+  install_name_tool -change "$OLD_LIBODBCINST_PATH" "@loader_path/libodbcinst.2.dylib" "$LIBMSODBCSQL_PATH"
 else
   echo "Warning: libodbcinst dependency not found in libmsodbcsql.18.dylib"
 fi
 
 if [ -n "$OLD_LIBLTDL_PATH" ] && [ -f "$LIBLTDL_PATH" ]; then
   echo "Fixing libodbcinst.2.dylib dependency on libltdl.7.dylib..."
   echo "  Changing: $OLD_LIBLTDL_PATH"
-  echo "  To: $LIBLTDL_PATH"
-  install_name_tool -change "$OLD_LIBLTDL_PATH" "$LIBLTDL_PATH" "$LIBODBCINST_PATH"
+  echo "  To: @loader_path/libltdl.7.dylib"
+  install_name_tool -change "$OLD_LIBLTDL_PATH" "@loader_path/libltdl.7.dylib" "$LIBODBCINST_PATH"
 else
   echo "Note: libltdl dependency not found or not needed"
 fi
 
 # Force codesign the dylibs
+
+# First set the IDs of the libraries using @loader_path
+echo "Setting library IDs with @loader_path..."
+echo "Setting ID for libmsodbcsql.18.dylib..."
+install_name_tool -id "@loader_path/libmsodbcsql.18.dylib" "$LIBMSODBCSQL_PATH"
+
+echo "Setting ID for libodbcinst.2.dylib..."
+install_name_tool -id "@loader_path/libodbcinst.2.dylib" "$LIBODBCINST_PATH"
+
+if [ -f "$LIBLTDL_PATH" ]; then
+  echo "Setting ID for libltdl.7.dylib..."
+  install_name_tool -id "@loader_path/libltdl.7.dylib" "$LIBLTDL_PATH"
+fi
+
 echo "Codesigning libmsodbcsql.18.dylib..."
 codesign -s - -f "$LIBMSODBCSQL_PATH" 2>/dev/null
 

mssql_python/libs/macos/lib/libodbcinst.2.dylib

@@ -1083,23 +1083,34 @@ ErrorInfo SQLCheckError_Wrap(SQLSMALLINT handleType, intptr_t handle, SQLRETURN
 std::wstring SanitizeConnectionString(const std::wstring& connectionString) {
     // This function will remove the UID and Pwd parameters for security reasons
     std::wstring sanitizedString = connectionString;
-    // Remove UID and Pwd parameters
-    size_t uidPos = sanitizedString.find(L"UID=");
+    std::wstring lowerCaseString = sanitizedString;
+    // Convert the string to lowercase for case-insensitive search
+    // Using lowerCaseString to avoid modifying the original string
+    // This is necessary because towlower works on wide characters
+    std::transform(lowerCaseString.begin(), lowerCaseString.end(), lowerCaseString.begin(),
+                   ::towlower);
+    // Can be UID or uid or UID, test only on lowercase uid
+    size_t uidPos = lowerCaseString.find(L"uid=");
     if (uidPos != std::wstring::npos) {
         size_t endPos = sanitizedString.find(L';', uidPos);
         if (endPos != std::wstring::npos) {
             sanitizedString.erase(uidPos, endPos - uidPos + 1);
+            lowerCaseString.erase(uidPos, endPos - uidPos + 1);
         } else {
             sanitizedString.erase(uidPos);
+            lowerCaseString.erase(uidPos);
         }
     }
-    size_t pwdPos = sanitizedString.find(L"Pwd=");
+    // Can be Pwd or pwd or PWD, test only on lowercase pwd
+    size_t pwdPos = lowerCaseString.find(L"pwd=");
     if (pwdPos != std::wstring::npos) {
         size_t endPos = sanitizedString.find(L';', pwdPos);
         if (endPos != std::wstring::npos) {
             sanitizedString.erase(pwdPos, endPos - pwdPos + 1);
+            lowerCaseString.erase(pwdPos, endPos - pwdPos + 1);
         } else {
             sanitizedString.erase(pwdPos);
+            lowerCaseString.erase(pwdPos);
         }
     }
     return sanitizedString;

mssql_python/pybind/CMakeLists.txt

@@ -2,7 +2,7 @@
 import sys
 from setuptools import setup, find_packages
 from setuptools.dist import Distribution
-from setuptools.command.bdist_wheel import bdist_wheel
+from wheel.bdist_wheel import bdist_wheel
 
 # Custom distribution to force platform-specific wheel
 class BinaryDistribution(Distribution):