Email for pocketbase

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

[Valexico]

Hi @joseph
So I did fork the pocketbase repo to fix the mail setting reset issue. Can I push a merge request somehow ?

[joseph]

@valexico yes, of course. https://git.cloudron.io/packages/pocketbase-app/ is the repo to create the MR against. Do you have a gitlab account? We can send you an invite for our git , should I send to this forum email? Otherwise, drop me a mail at support@cloudron.io with another email account if you want to keep it private.

[joseph]

@valexico but before you create the MR, does 9.1.7 not solve your problem? it's stable by now. Any reason not to add BREVO as mail relay in Cloudron and add it directly into pocketbase?

[Valexico]

Hi @joseph

I juste tested on a pocketbase app with cloudron 9.1.7. Sending a test email in the admin panel gives "Test email send timeout."

Anyway

• Nobody wants the sender name to be always reseted to "Pocketbase" so the Pocketbase package need a fix
• I want to be able to set a custom SMTP per app (such app can be deployed on a different domain for example). So i think defaulting mail setting to Cloudron SMTP is a expected, but overwriting custom user config on each reboot seems a bad pattern.

I'll be glad to propose a MR on this package, it's just a one line chante (moving the mail setting in the existing "first run section" of the start.sh)

You can invite me on this forum email