Email for pocketbase

[Valexico]

But this require some DNS update. As the mail server is already used for other app I think waiting for the fix seem more secure

[james]

Hello @Valexico
If the apps are on Cloudron, all should automatically update as well when you change the subdomain for the mail server.
But the RDNS will also need to be updated.
So I can understand why you are hesitant.

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