FEAT: Improved Connection String handling in Python by saurabh500 · Pull Request #307 · microsoft/mssql-python

added 6 commits

October 23, 2025 16:15

@saurabh500 saurabh500 changed the title Test PR FEAT: Improved Connection String handling in Python

Oct 29, 2025

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

github-advanced-security[bot]

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

@saurabh500

@saurabh500

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- 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

@saurabh500

@saurabh500

@saurabh500

- Moved _ConnectionStringAllowList class from connection_string_allowlist.py to constants.py
- Updated all imports in connection.py and test files

bewithgaurav

@saurabh500

jahnvi480

jahnvi480

jahnvi480

jahnvi480

jahnvi480

jahnvi480

@saurabh500

jahnvi480

- Added type hints to _ConnectionStringParser.__init__()
- Added type hints to _get_reserved_parameters()
…ogic and improving parameter normalization

@saurabh500

bewithgaurav

bewithgaurav

@saurabh500

bewithgaurav

jahnvi480

@saurabh500

@saurabh500 saurabh500 deleted the saurabh/connection_string_allow_list branch

November 7, 2025 07:50