Email Settings

[darren]

If I want to persist my own outgoing email server settings within the application, what's the best way to go about doing that? I can see that start.sh is overwriting email settings every time the app restarts.

[james]

Do you mean => https://docs.cloudron.io/apps/#disable-email-configuration?

But very good that you mention the start.sh.
I just took a peek https://git.cloudron.io/packages/limesurvey-app/-/blob/master/start.sh?ref_type=heads#L35-L44

echo "==> Configure email"
$mysql -e "REPLACE INTO lime_settings_global (stg_name, stg_value) VALUES ('siteadminemail', '${CLOUDRON_MAIL_FROM}')"
# support for single quote in splace name
display_name=$(echo -n "${CLOUDRON_MAIL_FROM_DISPLAY_NAME:-LimeSurvey}" | base64)
$mysql -e "REPLACE INTO lime_settings_global (stg_name, stg_value) VALUES ('siteadminname', FROM_BASE64('${display_name}'))"
$mysql -e "REPLACE INTO lime_settings_global (stg_name, stg_value) VALUES ('siteadminbounce', '${CLOUDRON_MAIL_FROM}')"
$mysql -e "REPLACE INTO lime_settings_global (stg_name, stg_value) VALUES ('emailmethod', 'smtp')"
$mysql -e "REPLACE INTO lime_settings_global (stg_name, stg_value) VALUES ('emailsmtpssl', '')"
$mysql -e "REPLACE INTO lime_settings_global (stg_name, stg_value) VALUES ('emailsmtphost', '${CLOUDRON_MAIL_SMTP_SERVER}:${CLOUDRON_MAIL_SMTP_PORT}')"
$mysql -e "REPLACE INTO lime_settings_global (stg_name, stg_value) VALUES ('emailsmtpuser', '${CLOUDRON_MAIL_SMTP_USERNAME}')"

There is no check surrounding this based on https://docs.cloudron.io/apps/#disable-email-configuration.
This means, the app needs to be updated to include such a check. Otherwise, even when disabling via. the UI this will still try to set it.
Thanks for reporting!

[darren]

@james yes I am also not sure this app has the option to disable email configuration even presented in the UI which would make sense since there's no check in the start.sh to respect a UI pref.

[james]

@BrutalBirdie did the changes https://git.cloudron.io/packages/limesurvey-app/-/merge_requests/25
Will be reviewed, and then the app update will be published.

[nebulon]

Updated package with optional sendmail is out now.

[darren]

Thank you!

[james]

@darren we can take the thanks as a "This now solved my issue", yes?
If so, you can just upvote this message.