Sequelize Connection string issues when there is no instanceName

[jurmb84]

My companies SQL Server is using the default instance name MSSQLSERVER.
In reviewing an issue I had connecting, I looking in msnodesqlv8/lib/sequelize/connection.js

If I set the instanceName option to MSSQLSERVER it is changed on line 58 to ''. However the connection string builder starting on line 60 is listed as:

config.connectionString = (config.driver ? `Driver={${config.driver}};` : '') +
        `Server=${config.server ? config.server : 'localhost'}\\${config.instanceName};` +
        (config.database ? `Database=${config.database};` : '') +
        (config.trustedConnection ? 'Trusted_Connection=yes;' : `Uid=${config.userName || ''};Pwd=${config.password || ''};`)

However, without an instance name the server is listed as Server=(server)\\; That is incorrect and causes issues. Also, encryption is not built into the connection string builder. I have changed it to the following and it is now working correctly.

config.connectionString = (config.driver ? `Driver={${config.driver}};` : '') +
        `Server=${config.server ? config.server : 'localhost'}` +
        (config.instanceName ? `\\${config.instanceName};` : ';') +
        (config.database ? `Database=${config.database};` : '') +
        (config.trustedConnection ? 'Trusted_Connection=yes;' : `Uid=${config.userName || ''};Pwd=${config.password || ''};`) +
        `Encrypt=${config.encrypt ? 'yes' : 'no'};`

Thank you for your attn to this matter.