npm audit fix failed

[SukeshP1995]

after installing using npm install while doing npm audit fix getting the followinf error

Expected behaviour:

Expected to run npm audit fix successfully

Actual behaviour:

npm audit fix failed

Software versions

• NodeJS: 15.11.0
• node-mssql: 6.2.3
• SQL Server: 2017

[ceisele-r]

Duplicate of #1288.

But I think the issue here is more and more people run into this and it will not go away because
node-mssql depends on adal-node that is archived (https://github.com/AzureAD/azure-activedirectory-library-for-nodejs) and will probably not receive updates to update to the new publish name of xmldom.
Therefore, I guess the better path forward would be to replace the use of adal-node here in node-mssql with @azure/msal-node (as it's stated in the readme of adal-node - but I don't know how complicated it is).

=> Then, as @azure/msal-node does not depend on xmldom, I think the audit warning would be gone for node-mssql.

[dhensby]

The issue being reported here is that npm audit fix is not fixing the problem - that sounds like a problem with npm not with this library.

[SukeshP1995]

I used same npm version 2 weeks back and the same node-mssql versiob 2 weeks back and it was working fine.

[SukeshP1995]

same configuration failed unexpectedly. Please, suggest a solution.

[dhensby]

The problem is with the tooling, not with the library. I can't provide a solution to npm not working, sorry.

[SukeshP1995]

what happens if we dont run npm audit --fix?

[dhensby]

I'm afraid this isn't a forum for providing support with npm/node - this is simply for reporting bugs with the mssql library.

Sorry I can't provide any more help

[SukeshP1995]

okay