Email Settings

[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.

[darren]

Yes thanks I think this is resolved!

[darren]

Hi @james I think there's been a regression. Email passwords are no longer persisted following app updates. I think other settings seem to be persisted.

[joseph]

@darren Can you explain more? Initial report was about optional sendmail which is now resolved. What do you mean by "Email passwords are no longer persisted" ? Do you mean you have sendmail disabled but for some reason email password (your custom setup) is not persisted?

[girish]

Great catch. @joseph there was some old migration workaround I added many years ago.

https://git.cloudron.io/packages/limesurvey-app/-/merge_requests/39