microsoft / go-sqlcmd

The new sqlcmd, CLI for SQL Server and Azure SQL (winget install sqlcmd / sqlcmd create mssql / sqlcmd open ads)
https://learn.microsoft.com/sql/tools/sqlcmd/go-sqlcmd-utility
MIT License
337 stars 59 forks source link

Various fixes from the bug bashes #253

Closed stuartpa closed 1 year ago

stuartpa commented 1 year ago

Fixes for:

https://github.com/microsoft/go-sqlcmd/issues/249 https://github.com/microsoft/go-sqlcmd/issues/246 https://github.com/microsoft/go-sqlcmd/issues/243

Implements a workaround for:

https://github.com/microsoft/go-sqlcmd/issues/257

which covers the wider issue of passing Credentials to ADS on Mac, which can't done at the moment. The workaround here in sqlcmd is to print to console the command that can be run to see the connection strings (which include the password), which the user can one time cut and paste into ADS (and select Save Password)

Adds test coverage (unit test coverage is now 86.9% (on Windows))