-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add support for Azure SQL Server (and Azure AD auth) #16
Comments
👍 Thanks for adding an issue. It is on our radar for sure, in fact we can use this issue to track it. Not sure when we'll get to it, but it's an important use case we want to support. |
Looks like this will require moving our mssql implementation to tediusjs instead http://tediousjs.github.io/tedious/getting-started.html |
Quite possibly; I believe that is what is used for a vscode mssql extension i was using as well, So there's basis for that direction. |
@ransagy I think this was fixed a while ago right? You should be able to use AD credentials to log in now. Please close if so |
@rathboma I grabbed the latest released portable and tried again; Since the AAD username is an email, It thinks im trying to connect to the host part of the email as a domain and fails. So i don't think it works right now, Unless im missing some specific way to indicate its Azure SQL. However, I CAN connect using the serveradmin sql account that was created when the Azure SQL instance was made; But i don't think that's the case either of us were thinking about. |
I'd really like to give this app a good run in my day to day workload.
Most of the time, I'm working on SQL Server instances in our Azure subscription with AD/MFA authentication to use my Azure account to log in.
Please consider adding support for this connection scenario.
Thanks!
The text was updated successfully, but these errors were encountered: