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

[Bug]: Default MSSQL username is sa not SA #7237

Closed
Eng-Fouad opened this issue Jun 22, 2023 · 1 comment · Fixed by #7238
Closed

[Bug]: Default MSSQL username is sa not SA #7237

Eng-Fouad opened this issue Jun 22, 2023 · 1 comment · Fixed by #7238
Labels

Comments

@Eng-Fouad
Copy link
Contributor

Eng-Fouad commented Jun 22, 2023

Module

MSSQLServer

Testcontainers version

1.18.3

Using the latest Testcontainers version?

Yes

Host OS

Any

Host Arch

Any

Docker version

24.0.2

What happened?

If MSSQL_COLLATION is set to a collation with case-sensetive (_CS not _CI), the username SA will fail to login to DB, as sa is the correct username.

mssqlServerContainer.addEnv("MSSQL_COLLATION", "Arabic_100_CS_AS_KS_WS_SC_UTF8");

Relevant log output

No response

Additional Information

No response

@eddumelendez
Copy link
Member

Hi, thanks for raising the issue. But, no need to do it if a PR is coming afterwards.

@eddumelendez eddumelendez closed this as not planned Won't fix, can't repro, duplicate, stale Jun 22, 2023
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 a pull request may close this issue.

2 participants