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

Login failed for user ''. on v9.1.0 #1460

Closed
aterna01 opened this issue Jan 18, 2023 · 3 comments
Closed

Login failed for user ''. on v9.1.0 #1460

aterna01 opened this issue Jan 18, 2023 · 3 comments

Comments

@aterna01
Copy link

aterna01 commented Jan 18, 2023

Can not connect to SQL server

Expected behaviour:

Successfully connect to SQL server

Connect to SQL and successfully run queries

Actual behaviour:

Get an error when trying to connect to SQL. This happens on the latest v9.1.0 but works fine on v9.0.1

This is the function which is called to connect to SQL

async function getSqlServerDriver() {
const pool = new sql.ConnectionPool(config.sql.connectionString);
await pool.connect();
return pool;
}

The error is

Login failed for user ''.
      at Object.onceWrapper (node:events:628:26)
      at Connection.emit (node:events:513:28)
      at Connection.emit (node:domain:489:12)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)

Configuration:

// paste relevant config here
config.sql.connectionString = "Server=x.x.x.x;Database=someDb;User Id=someUser;Password=somePass;trustServerCertificate=true;request timeout=30000"

Software versions

  • NodeJS: v16.18.1
  • node-mssql: v9.1.0
  • SQL Server: Microsoft SQL Server 2019 (RTM) - 15.0.2000.5 (X64)
@heikkilamarko
Copy link

We faced this same problem after updating from v9.0.1 to v9.1.0.

@dhensby
Copy link
Collaborator

dhensby commented Jan 18, 2023

I'll investigate, thanks for reporting

@dhensby
Copy link
Collaborator

dhensby commented Feb 15, 2023

This regression was fixed in 9.1.1

@dhensby dhensby closed this as completed Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants