Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(mssql): use integrated auth when no user or password supplied #8668

Merged
merged 4 commits into from
Mar 17, 2024

Conversation

gforsyth
Copy link
Member

The method for enabling Integrated Authentication with ODBC is to not
provide a user or password, and to set Trusted_Connection = 'true'.

If Trusted_Connection is already set in kwargs, do nothing, because
either the user set it to 'true' and it will work untouched, or they're
specifying 'false' for a reason and we should leave it alone.- feat(mssql): use integrated auth when no user or password supplied

  • docs(mssql): add odbc tabset and integrated authentication section

Resolves #8657

Copy link
Member

@jcrist jcrist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM! Two small comments, feel free to ignore all.

ibis/backends/mssql/__init__.py Outdated Show resolved Hide resolved
docs/backends/mssql.qmd Outdated Show resolved Hide resolved
gforsyth and others added 4 commits March 17, 2024 07:56
The method for enabling Integrated Authentication with ODBC is to not
provide a `user` or `password`, and to set `Trusted_Connection =
'true'`.

If `Trusted_Connection` is already set in `kwargs`, do nothing, because
either the user set it to 'true' and it will work untouched, or they're
specifying 'false' for a reason and we should leave it alone.
Added a tabset to show the different probable driver strings required
for using ODBC vs unixODBC.
Documented the new defaults for integrated authentication.
Co-authored-by: Jim Crist-Harif <jcristharif@gmail.com>
@cpcloud cpcloud force-pushed the mssql_integrated_auth branch from 0a90107 to 6f697a9 Compare March 17, 2024 11:56
@cpcloud cpcloud added docs Documentation related issues or PRs mssql The Microsoft SQL Server backend labels Mar 17, 2024
@cpcloud cpcloud enabled auto-merge (squash) March 17, 2024 11:57
@cpcloud cpcloud added this to the 9.0 milestone Mar 17, 2024
@cpcloud cpcloud merged commit 0a78414 into ibis-project:main Mar 17, 2024
89 of 91 checks passed
@gforsyth gforsyth deleted the mssql_integrated_auth branch March 19, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related issues or PRs mssql The Microsoft SQL Server backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(mssql): Explicit Integrated Auth for MSSQL Connections
3 participants