Email for pocketbase

[Valexico]

hi @james
New issue here
As a workaround I used a third party SMTP (BREVO)
But when pocketbase restart it is set back to Cloudron SMTP. IS this expected behaviour ? I think this is not a good practice you should allow custom app configuration here ?

https://git.cloudron.io/packages/pocketbase-app/-/blob/main/start.sh?ref_type=heads#L33

This should not be in start.sh it should run only on fresh install and then let the user customize

[james]

Hello @valexico
Some apps we package with optional mail setup configuration that can be disabled in the dashboard.
This app is currently non-optional.
We started to make a difference for apps that only send transactional mail and apps that do not.
PocketBase to our knowledge only sends transactional mails, for what the Cloudron Mail server is perfect.

Can you tell me why you'd like to configure a different mailer for this app specifically?

[Valexico]

@james For 2 reason

• Because at the moment I am still waiting for the Cloudron update allowing to send mails with default configuration (I don't want to mess my working DNS configuration)
• Because for customer facing apps, I think I may have a higher delivery rate with transactional mail services (currently my mails at tagged SPAM on some users)

I think the second reason is valid. But I found a workaround : as this app have a custom domain I can configure an external SMTP for this domain. My only issue currently is the first one (which should be solved soon if I understand)

[Valexico]

@james Even the "sender name" is overwritten on each reboot. I think this is too much as you are supposed to customize the app name here

[Valexico]

I think the update_config should just be moved in the if bloc it would make more sense :

[joseph]

@Valexico you can update to 9.1.7 which fixes the email sending issue. Can you please check/confirm here please?

[Valexico]

Thanks, I'll check on Monday, but my comment stay valid : I don't want the sender name to always be set back to "Pocketbase". This will not speak to the app users which expect mail sender to be the app name

[joseph]

@Valexico yeah, I think that's a packaging issue. We will look into it.

[Valexico]

Hi @joseph

• So 9.1.7 is marked "This update is a pre-release and not considered stable yet. Please update at your own risk." So I didn't update yet
• For pocketbase, I had to fork the repo. The reset of the email config on each restart is not acceptable for production.

Waiting for a packaging update of the official package

[joseph]

@valexico 9.1.7 is being rolled out. When it's being rolled out, we mark is as unstable , so that everyone does not update all together .