go-sqlcmd

go-sqlcmd copied to clipboard

In implementing `sqlcmd open ads` for Mac, we've hit an issue, in that the name of the credential that ADS is expecting (via SqlToolsService) is named using UTF16 characters (the...

stuartpa

Generated connection strings are incorrect

9

When generating the connection strings via `sqlcmd config connection-strings`, the ADO.NET doesn't work as is: - The connection string contains spaces that make it hard to just copy-and-paste it into...

yorek

would be helpful to be able to modify a user/endpoint/context rather than delete/re-create

bweissman

sqlcmd open ads It prompts error :  Expected Output : Would be nice to have an equivalent of the sqlcmd -C, --trust-server-certificate, on the azuredatastudio.exe cmd line, so...

JyotikaGargg

Debug messages seen with default verbosity

1

I had docker instance created along with explicitly installed sql server and when trying to connect to docker instance using backward compat syntax I see these messages. ``` C:\Users\apdeshmukh\git\go-sqlcmd>.\sqlcmd.exe -Slocalhost:1435...

apoorvdeshmukh

Currently the Testcmd function just executes a command, but doesn't allow for further verification that the output generated was expected cmdparser.TestSetup(t) cmdparser.TestCmd[*AddEndpoint]() cmdparser.TestCmd[*AddContext]("--endpoint endpoint") cmdparser.TestCmd[*UseContext]("--name context") Provide a pattern that...

stuartpa

Ensure sqlcmd query doesn't return error after sqlcmd start

2

Right now `sqlcmd start` does not block waiting for SQL Server to be running and ready for client connections. So if `sqlcmd query` is immediately run, it will fail with:...

stuartpa

For POSIX systems, a man(1) page should be provided. The README.md already looks like it might have a good starting point for the man page content.

ascarter

Enable user to override the default application name sqlcmd puts in the connection string

2

This is related to the other issue on application_name. Where we are looking to set application name to "sqlcmd", but this request go further and asks that the application name...

stuartpa

Pattern used for catching expected errors in tests should be able to verify for precise expected error

1

I've been using this pattern for negative tests, that are expecting a failure: defer func() { test.CatchExpectedError(recover(), t) }() This pattern ends up causing a test to pass for any...

stuartpa