Skip to content

Commit b247063

Browse files
authored
Merge branch 'main' into subrata-ms/SegFault
2 parents ae23203 + fb7b5c3 commit b247063

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

mssql_python/auth.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -165,8 +165,7 @@ def remove_sensitive_params(parameters: List[str]) -> List[str]:
165165
exclude_keys = [
166166
"uid=",
167167
"pwd=",
168-
"encrypt=",
169-
"trustservercertificate=",
168+
"trusted_connection=",
170169
"authentication=",
171170
]
172171
result = [

tests/test_008_auth.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -309,15 +309,17 @@ def test_remove_sensitive_parameters(self):
309309
"Encrypt=yes",
310310
"TrustServerCertificate=yes",
311311
"Authentication=ActiveDirectoryDefault",
312+
"Trusted_Connection=yes",
312313
"Database=testdb",
313314
]
314315
filtered_params = remove_sensitive_params(params)
315316
assert "Server=test" in filtered_params
316317
assert "Database=testdb" in filtered_params
317318
assert "UID=user" not in filtered_params
318319
assert "PWD=password" not in filtered_params
319-
assert "Encrypt=yes" not in filtered_params
320-
assert "TrustServerCertificate=yes" not in filtered_params
320+
assert "Encrypt=yes" in filtered_params
321+
assert "TrustServerCertificate=yes" in filtered_params
322+
assert "Trusted_Connection=yes" not in filtered_params
321323
assert "Authentication=ActiveDirectoryDefault" not in filtered_params
322324

323325

0 commit comments

Comments
 (0)