FEAT: Improved Connection String handling in Python by saurabh500 · Pull Request #307 · microsoft/mssql-python
added 6 commits
October 23, 2025 16:15
saurabh500
changed the title
Test PR
FEAT: Improved Connection String handling in Python
Saurabh Singh (SQL Drivers) added 3 commits
October 29, 2025 06:22…ers and improving exception management
…nd normalizing test cases
…d updating filtering logic
Saurabh Singh (SQL Drivers) added 4 commits
October 29, 2025 14:16… to _ConnectionStringAllowList - Updated references in connection.py, connection_string_allowlist.py, connection_string_parser.py, and related test files to use the new class name. - Changed method names from `parse` to `_parse` in _ConnectionStringParser for internal consistency. - Adjusted unit tests to reflect the new class and method names, ensuring all tests pass with the updated structure. - Improved code clarity and maintainability by adhering to naming conventions for internal classes.
…nd improve error handling for empty values
…ate test cases for parameter normalization
… include synonym and clarify usage
Copilot AI review requested due to automatic review settings
October 30, 2025 23:16- Removed unused 'List' and 'logging' imports from connection_string_parser.py - Removed unused 'sys', 'pooling', and 'threading' imports from test_003_connection.py - Removed unused variables and unnecessary pass statements in tests - Cleaned up test_012_connection_string_integration.py imports All tests passing: 226 passed, 4 skipped
- Moved _ConnectionStringAllowList class from connection_string_allowlist.py to constants.py - Updated all imports in connection.py and test files
- Added type hints to _ConnectionStringParser.__init__() - Added type hints to _get_reserved_parameters()
saurabh500
deleted the
saurabh/connection_string_allow_list
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters