-
Notifications
You must be signed in to change notification settings - Fork 765
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
Create custom user account on run #49
Comments
FYI - We released CTP 1.4 today. This release of the mssql-server-linux image now includes the mssql-tools package (sqlcmd and bcp) in it. Executing sqlcmd as part of the entrypoint.sh script can be used for this kind of scenario for now. Since this is such a commonplace requirement we want to make it easier in the future, but sqlcmd will provide a reasonable option until then. |
Hi, I tried to use the tag CTP 1.4 but sqlcmd is not found in terminal. Thanks |
Hi, I saw your demo https://github.com/twright-msft/mssql-node-docker-demo-app, and i was able to use the sqlcmd base on your guide. Maybe we could include this code in creating the docker image, in order for the sqlcmd be visible globally? |
We've run into problems in the past with conflicting file names, in particular the rather generic name 'bcp'. So, for now at least, we recommend pointing to the tools using the fully qualified path. |
This is my Dockerfile for building a custom image based on the ctp1-1: RUN apt-get update && |
OTOH, why not make the image users even easier MySQL images, for example, are configurable allow to create extra user via environment
Why not provide similar hook into the mssql image initialization? |
I don't think anyone is looking into this issue anymore. |
@pawelmaslyk They should have closed it then. @twright-msft? |
We still plan on adding this. Just havent gotten to it yet. |
Any update on this please |
Any news on this? |
From the comments section in DockerHub: https://hub.docker.com/r/microsoft/mssql-server-linux/
Create a customer user admin login with environment variables for added security.
The text was updated successfully, but these errors were encountered: