Replies: 7 comments 5 replies
|
I have found this to be true also, I actually removed this PR (and a few others) and rebased the image of louislam's new 1.5.0 release...just haven't updated this github yet. tag is |
|
This particular PR technically goes against @louislam 's style but that is the nice thing about forks, you can do whatever (just may never be merged back to official) Looking into it further, PR louislam#426 does not make the server read USER/PASSWORD variables on startup. That said, the way around this would be to add a other method would be to create a start script and add it to the Dockerfile and build a new image that does the same thing and creates some type of marker file so as to run only once if that marker file is detected ensuring it only runs once (until that file is removed). This however also isn't secure due to the password now being in the compose or env file...so then docker secrets would need to be used to try and keep things more secure. |
|
...just to mess around with the idea, I created a local branch with concepts of PR# 426 merged and tested (not extensively) and seems to work... I will probably not push the build image to the registry...but if you want I can push the branch to my git and from there you can clone the repo and build/run it yourself |
|
HI,
Ok thanks can you push the branch and tell me how to create the docker
image please ?
Best regards.
…On Thu, Oct 16, 2025 at 1:31 AM cmcooper1980 ***@***.***> wrote:
...just to mess around with the idea, I created a local branch with
concepts of PR# 426 merged and tested (*not extensively*) and seems to
work...
I will probably not push the build image to the registry...but if you want
I can push the branch to my git and from there you can clone the repo and
build/run it yourself
—
Reply to this email directly, view it on GitHub
<#14 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASWWRPHY5YHLQEDD72ZGTQT3X3KOPAVCNFSM6AAAAAB2GUM5EGVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTINRZGIZTQMA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Reynald CHEKHINA
*SA, Square1*
*e: ***@***.*** ***@***.***>*
Company Registration Number: 528714
VAT number IE3178050PH
|
|
HI, I have created the docker image and test it like following: it works and I dont need to put the env variable on the docker-compose.yml or .env file. Can you merge it please ? Best regards. |
|
HI, I mean that i execute the docker command so I dont need to have the env variables.
Best regards. |
|
HI,
yes sure. will you merge this branch and generate new docker image version ?
Best regards.
…On Tue, Oct 21, 2025 at 10:23 AM cmcooper1980 ***@***.***> wrote:
OK great, then you just delete/omit/comment out the variables in the
compose/.env file as noted in the 3rd option I mentioned previously
—
Reply to this email directly, view it on GitHub
<#14 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASWWRPGI6C2IIRAAPDDID6L3YXUQZAVCNFSM6AAAAAB2GUM5EGVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTINZTG43DGMY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Reynald CHEKHINA
*SA, Square1*
*e: ***@***.*** ***@***.***>*
Company Registration Number: 528714
VAT number IE3178050PH
|

Uh oh!
There was an error while loading. Please reload this page.
HI,
The PR louislam#426 doesnt work. If you set the env variables USER and PASSWORD the user is not created.
Can you investigate it please?
Best regards.
All reactions