Email for pocketbase

[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