xo / usql

Universal command-line interface for SQL databases
MIT License
8.88k stars 347 forks source link

Update go-mssqldb driver reference #365

Closed burningalchemist closed 2 years ago

burningalchemist commented 2 years ago

Notable changes:

With this change we only replace the driver and add azuresql alias for compatibility with xo and dburl modules.

To connect to MSSQL DB via Azure AD we need to import the submodule azuread. We could potentially replace microsoft/go-mssqldb with microsoft/go-mssqldb/azuread as it implicitly imports the root module and doesn't seem to have any bad implications. However, I'm not sure if it's a good idea, or we'd better treat azuresql differently.

It also might be a separate PR to keep the scope small.

References:

nineinchnick commented 2 years ago

Looks like this has been merged in c39cb6c2b431dcc2f223ff16efa130212c438241