Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[15.0, 16.0] Removed mail default "From" feature? (params: mail.force.smtp.from, mail.dynamic.smtp.from) #5895

Open
bobslee opened this issue Nov 17, 2022 · 1 comment
Assignees

Comments

@bobslee
Copy link

bobslee commented Nov 17, 2022

Impacted versions: 15, 16

Documentation doesn't match the Odoo implementation/code anymore, since version 15.
This isn't an issue (ignore) for version 14, 13 (and maybe earlier).

According to documentation:

Both config params mail.force.smtp.from, mail.dynamic.smtp.from can be used to provide/override the outbound sender (From) address.

When obtaining and setup according to the documentation (links above), the outbound mails still result in error which the (documented) feature solved in version 14 and 13.

(example mail addresses non real-world):

SMTPRecipientsRefused: {'[email protected]': (553, b'5.7.1 <[email protected]>: Sender address rejected: not owned by user [email protected]')}

The implementation (code) exists in version 14, but not in version 15 and 16 (just search the params).
See code in version 14 (which doesn't exist in 15, 16):

  • Model: ir.mail_server
  • Function: _get_email_from

Currently: https://github.com/odoo/odoo/blob/14.0/odoo/addons/base/models/ir_mail_server.py#L356

@thomas15v
Copy link

This seems fixed in the documentation, it is still very confusing to implement. But I think the new feature is more dynamic then just setting a forced parameter.

To get the same behavior as with mail.force.smtp.from you set mail.default.from and mail.catchall.domain correctly. And as last step you make sure your outgoing email configuration has a proper from filter. Odoo will then look around and set the email with the default email address.

@vava-odoo vava-odoo transferred this issue from odoo/odoo Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants