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

wait-for-it.sh has issues with IPv6 literals #252

Open
juliaszone opened this issue May 30, 2024 · 1 comment
Open

wait-for-it.sh has issues with IPv6 literals #252

juliaszone opened this issue May 30, 2024 · 1 comment
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@juliaszone
Copy link

Hello,
I was trying to setup a roundcubemail-docker container, but I saw it won't start up. Apparently the database "is not up".
The script used does not support IPv6 literals and considers the first colon as a port separator
See: vishnubob/wait-for-it#124
My proposal is to either move to a different script or add an ENVVAR to disable it.

roundcube_1  | roundcubemail not found in /var/www/html - copying now...
roundcube_1  | Complete! ROUNDCUBEMAIL has been successfully copied to /var/www/html
roundcube_1  | wait-for-it.sh: waiting 30 seconds for [2a01:4f8
roundcube_1  | wait-for-it.sh: timeout occurred after waiting 30 seconds for [2a01:4f8
roundcube_1  | Write root config to /var/www/html/config/config.inc.php
roundcube_1  | Write Docker config to /var/www/html/config/config.docker.inc.php
roundcube_1  | ERROR: SQLSTATE[08006] [7] invalid integer value "4f8" for connection option "port"
roundcube_1  | ERROR: Failed to connect to database
roundcube_1  | Failed to initialize/update the database. Please start with an empty database and restart the container.
roundcube_1  | Generating locales (this might take a while)...
roundcube_1  |   en_US.UTF-8... done
roundcube_1  | Generation complete.
roundcube_1  | [30-May-2024 06:51:30] NOTICE: fpm is running, pid 1
roundcube_1  | [30-May-2024 06:51:30] NOTICE: ready to handle connections
@pabzm pabzm added bug Something isn't working help wanted Extra attention is needed labels Nov 7, 2024
@pabzm
Copy link
Member

pabzm commented Nov 8, 2024

IPv6 should of course be supported. Anyone maybe motivated to propose a fix in a PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants