search

Maescool / docker-Cloudlog

Dockerized Cloudlog amateur radio logging application
20 stars 6 forks source link

New Install - QRZ logins not parsed #7

Closed Wireheadbe closed 10 months ago

Wireheadbe commented 10 months ago

Hi fellow OM,

Seems like with the latest builds, the preloads script isn't parsing the QRZ username and password isn't added to the config.php. Something is going wrong with the sed inline replace :)

Kind regards, ON8AD

Wireheadbe commented 10 months ago

The preload contains a typo: "QRZ_PASWORD" instead of "QRZ_PASSWORD" as given in the app env file.

Somehow, the environment variables where there (as verified by executing "export" in the docker container). However, the config wasn't getting those correct config variables for callbook / qrz username / qrz password. So I modified it temporary to this, and it seems to work. Dirty? Yes! Does it work? Yes! :) 

sed -ri "/callbook/c \$config[\'callbook\'] = \"${CALLBOOK}\"\;" ${CONFIGFILE}
sed -ri "/qrz_username/c \$config[\'qrz_username\'] = \"${QRZ_USERNAME}\"\;" ${CONFIGFILE}
sed -ri "/qrz_password/c \$config[\'qrz_password\'] = \"${QRZ_PASSWORD}\"\;" ${CONFIGFILE}
Wireheadbe commented 10 months ago

made a pull request with some updated sed statements

Maescool commented 10 months ago

Thank you for the fixes 73 ON1GPS