Unable to start, error: port "" is not a valid integer #80

Closed
opened 2025-11-20 04:19:56 -05:00 by saavagebueno · 4 comments
Owner

Originally created by @xaionaro on GitHub (Dec 13, 2020).

user@dx-ddns:~/go/bin$ ./updater 
=========================================
=========================================
========= DDNS Universal Updater ========
=========================================
=== Made with ❤️  by github.com/qdm12 ====
=========================================

Running version ? built on ? (commit ?)


🔧  Need help? https://github.com/qdm12/ddns-updater/issues/new
💻  Email? quentin.mcgaw@gmail.com
☕  Slack? Join from the Slack button on Github
💸  Help me? https://github.com/sponsors/qdm12
{"level":"ERROR","ts":"2020-12-13T13:38:30.425Z","msg":"port \"\" is not a valid integer"}

In strace -f ./updater I do not see any attempts to read any json file.

Originally created by @xaionaro on GitHub (Dec 13, 2020). ``` user@dx-ddns:~/go/bin$ ./updater ========================================= ========================================= ========= DDNS Universal Updater ======== ========================================= === Made with ❤️ by github.com/qdm12 ==== ========================================= Running version ? built on ? (commit ?) 🔧 Need help? https://github.com/qdm12/ddns-updater/issues/new 💻 Email? quentin.mcgaw@gmail.com ☕ Slack? Join from the Slack button on Github 💸 Help me? https://github.com/sponsors/qdm12 {"level":"ERROR","ts":"2020-12-13T13:38:30.425Z","msg":"port \"\" is not a valid integer"} ``` In `strace -f ./updater` I do not see any attempts to read any json file.
Author
Owner

@qdm12 commented on GitHub (Dec 13, 2020):

Are you trying to run the binary out of the container? If so you need to set a few environment variables, you can have a look at the Dockerfile ENV instructions.

LISTENING_PORT=8000 updater

Should fix that one. If it complains more, you can adjust environment variables. I'll work on instructions to make it runnable out of a container today so let's leave the issue opened.

@qdm12 commented on GitHub (Dec 13, 2020): Are you trying to run the binary out of the container? If so you need to set a few environment variables, you can have a look at the Dockerfile `ENV` instructions. ```sh LISTENING_PORT=8000 updater ``` Should fix that one. If it complains more, you can adjust environment variables. I'll work on instructions to make it runnable out of a container today so let's leave the issue opened.
Author
Owner

@xaionaro commented on GitHub (Dec 13, 2020):

Are you trying to run the binary out of the container?

Yes.

If so you need to set a few environment variables, you can have a look at the Dockerfile ENV instructions.

It helped. Thank you.

IMHO, the main problem here is unclear error reporting. It makes sense to mention LISTENING_PORT in the error message.

@xaionaro commented on GitHub (Dec 13, 2020): > Are you trying to run the binary out of the container? Yes. > If so you need to set a few environment variables, you can have a look at the Dockerfile ENV instructions. It helped. Thank you. IMHO, the main problem here is unclear error reporting. It makes sense to mention `LISTENING_PORT` in the error message.
Author
Owner

@qdm12 commented on GitHub (Dec 13, 2020):

Yep actually that's because of a dependency (qdm12/golibs) I got lazy at upgrading, will do it today 😅 Thanks for the feedback!

@qdm12 commented on GitHub (Dec 13, 2020): Yep actually that's because of a dependency (qdm12/golibs) I got lazy at upgrading, will do it today 😅 Thanks for the feedback!
Author
Owner

@qdm12 commented on GitHub (Dec 13, 2020):

Alright I'm upgraded the dependencies, so now error messages should be clearer. I'll skip adding documentation to run it as a binary for now as you need the ui files as well for now, and this was designed especially to run as a container. Thanks!

@qdm12 commented on GitHub (Dec 13, 2020): Alright I'm upgraded the dependencies, so now error messages should be clearer. I'll skip adding documentation to run it as a binary for now as you need the ui files as well for now, and this was designed especially to run as a container. Thanks!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ddns-updater#80