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

V1.18.0 seems to break the extension #17644

Closed
lewismazzei opened this issue Mar 24, 2023 · 7 comments
Closed

V1.18.0 seems to break the extension #17644

lewismazzei opened this issue Mar 24, 2023 · 7 comments

Comments

@lewismazzei
Copy link

  • MSSQL Extension Version: V1.18.0
  • VSCode Version: 1.76.2
  • OS Version: 13.0

Steps to Reproduce:

  1. MS SQL: Execute query
    mssql: Error: Login failed. Retry using different credentials?
  2. Retry:
    mssql: Connection error : Login failed for user 'SA'.
    and
    mssql: Failed to connect: System.ArgumentNullException: Value cannot be null. (Parameter 'User ID')
    at Microsoft.Data.Common.ADP.CheckArgumentNull(Object value, String parameterName)
    at Microsoft.Data.SqlClient.SqlConnectionStringBuilder.SetValue(String keyword, String value)
    at Microsoft.Data.SqlClient.SqlConnectionStringBuilder.set_UserID(String value)
    at Microsoft.SqlTools.ServiceLayer.Connection.ConnectionService.CreateConnectionStringBuilder(ConnectionDetails connectionDetails) in D:\a_work\1\s\src\Microsoft.SqlTools.ServiceLayer\Connection\ConnectionService.cs:line 1417
    at Microsoft.SqlTools.ServiceLayer.Connection.ConnectionService.BuildConnectionString(ConnectionDetails connectionDetails) in D:\a_work\1\s\src\Microsoft.SqlTools.ServiceLayer\Connection\ConnectionService.cs:line 1374
    at Microsoft.SqlTools.ServiceLayer.Connection.ConnectionService.TryOpenConnection(ConnectionInfo connectionInfo, ConnectParams connectionParams) in D:\a_work\1\s\src\Microsoft.SqlTools.ServiceLayer\Connection\ConnectionService.cs:line 674
@kburtram
Copy link
Member

@cheenamalhotra any ideas what could be happening here or how to investigate?

@lewismazzei have you tried deleting the connection and readding it? Sometimes that can clear out "data state" about a particular connection.

@cheenamalhotra
Copy link
Member

@lewismazzei

Since it says Login failed for user 'SA' - I believe this is 'SqlLogin' authentication mode.
When you were prompted with "Login failed. Retry using different credentials?" what details did you fill in? Can you find your connection profile from settings.json and let us know what options are included?

@markusjohnsson
Copy link

I'm getting this too. Connecting to azure db using connection string. Firewall includes my ip. Exact connection string works from separate .net app. I'm on macOS 13.2.

@scharnyw
Copy link

scharnyw commented Apr 27, 2023

Same here. I'm connecting to a SQL server database SaaS on Azure using connection string. On 1.18 I'm getting the error:

mssql: Connection error : Login failed for user 'xxxx'.

No error logs in VS Code extension host, and in the MSSQL extension logs for that session it shows:

[5:14:03 PM] Connecting to server "{serverEndpoint}" on document "{connectionString}".
[5:14:06 PM] Connected to server "{serverEndpoint}" on document "{connectionString}". Server information: bla bla bla

Yet I'm seeing the error on UI. After downgrading to 1.17.1 everything works again.

I'm on Windows and using VS Code 1.77.3.

@markusjohnsson
Copy link

Still unusable here.

@cheenamalhotra cheenamalhotra added this to the July 2023 Release milestone May 18, 2023
@Henningsson
Copy link

unusable here on osx

@cheenamalhotra
Copy link
Member

Issue has been addressed and verified - fix will be included in upcoming extension release - v1.20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants