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

Adapt to mssql-docker breaking change #6484

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Aug 7, 2024

2 things need to change:

  • the path to the binary, which is now secure by default;
  • the addition of the -C flag, telling sqlcmd to trust the certificate of the server.

Quote from the documentation:

In SQL Server 2022 (16.x) CU 14 and later versions, container images
include the new mssql-tools18 package. The previous directory
/opt/mssql-tools/bin is being phased out. The new directory for
Microsoft ODBC 18 tools is /opt/mssql-tools18/bin, aligning with the
latest tools offering. For more information about changes and security
enhancements, see https://techcommunity.microsoft.com/t5/sql-server-blog/odbc-driver-18-0-for-sql-server-released/ba-p/3169228

See microsoft/mssql-docker#892

2 things need to change:

- the path to the binary, which is now secure by default;
- the addition of the -C flag, telling sqlcmd to trust the certificate
  of the server.

Quote from the documentation:

> In SQL Server 2022 (16.x) CU 14 and later versions, container images
> include the new mssql-tools18 package. The previous directory
> /opt/mssql-tools/bin is being phased out. The new directory for
> Microsoft ODBC 18 tools is /opt/mssql-tools18/bin, aligning with the
> latest tools offering. For more information about changes and security
> enhancements, see https://techcommunity.microsoft.com/t5/sql-server-blog/odbc-driver-18-0-for-sql-server-released/ba-p/3169228

See microsoft/mssql-docker#892
@greg0ire greg0ire added the CI label Aug 7, 2024
@greg0ire greg0ire added this to the 3.8.7 milestone Aug 7, 2024
@greg0ire greg0ire merged commit a456fbf into doctrine:3.8.x Aug 7, 2024
80 checks passed
@greg0ire greg0ire deleted the try-new-path branch August 7, 2024 11:26
@derrabus
Copy link
Member

derrabus commented Aug 7, 2024

Thank you!

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

Successfully merging this pull request may close these issues.

3 participants