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

Update postfix.sh, so mysql_sender_dependent_default_transport_maps.cf is in line with mysql_sasl_passwd_maps_sender_dependent.cf #6085

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from

Conversation

freimuts
Copy link

What does this PR include?

Short Description

In case when the domain has relay host configures (rh.example.com) and you configure a separate relay host for user within that domain (rh2.example.com) credentials for relay host are selected from domain relay host (rh.example.com) no users, like:

[at]example.com - relay host rh.example.com
example[at]example.com - relay host rh2.example.com

Expected result:
When sending email from any email under domain use rh.example.com, when sending from example[at]example.com use rh2.example.com, both hosts require authentication, correct authentication is provided and email sent.

Actual result:
When sending email from any email under domain use rh.example.com, when sending from example[at]example.com use rh2.example.com, authentication credentials for rh2.example.com are selected from rh.example.com.

Cause:
mysql_sender_dependent_default_transport_maps.cf SELECT COALESCE starts by relayhosts LEFT OUTER JOIN mailbox and then uses relayhosts LEFT OUTER JOIN domain, but mysql_sasl_passwd_maps_sender_dependent.cf currently does opposite, so I flipped the order so that both queries have same logic.

Affected Containers

  • postfix

Did you run tests?

Yes

What did you tested?

I tested postfix, sending, receiving mail using relays and not, relays in different configurations. Since I have used this fix in production for about two weeks now.

What were the final results? (Awaited, got)

The connection was always made and authenticated correctly.

@freimuts freimuts changed the title Update postfix.sh Update postfix.sh, so mysql_sender_dependent_default_transport_maps.cf is in line with mysql_sasl_passwd_maps_sender_dependent.cf Sep 19, 2024
@DerLinkman DerLinkman force-pushed the staging branch 2 times, most recently from aaee040 to 4d688c5 Compare November 15, 2024 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants