It looks to me that the commit mentioned just uses a manually configured connection string to get the tests to work (trustServerCertificate=true) and does not solve the problem with the extension itself, that you can’t create a datasource in the admin that will connect using a version 10+ MSSQL driver.

I checked this issue with lucee version 6.0.0.432-SNAPSHOT. When I tried to create a datasource connection with MSSQL extension version 12.2.0.jre8, it didn't cause any error and created the datasource successfully. The issue is solved.
This commit solved this issue https://github.com/lucee/Lucee/commit/e574eb9d63b8473dd89259339ddcf48c33519fca

I believe that this is due to a change in the MS drivers affecting version 10+ where the setting encrypt now defaults to true.

The admin fields that are used to define the datasource do not include this (and other) settings.

Maybe a field could be added that lets additional connection parameters be added as a string. That way the admin fields do not have to keep step with the drivers. I think that ACF does this.

https://dev.lucee.org/t/connecting-to-mssql-server-using-the-latest-drivers/12347/3?u=martin