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

TLSRPT follow-up updates #112

Merged
merged 3 commits into from
Feb 4, 2025
Merged

TLSRPT follow-up updates #112

merged 3 commits into from
Feb 4, 2025

Conversation

Snawoot
Copy link
Owner

@Snawoot Snawoot commented Feb 4, 2025

Purpose of proposed changes

  • Makes TLSRPT reporting configurable on per-zone basis.
  • Send mx_host_pattern attributes along with policy response.

Related #107

Essential steps taken

Moved config property to zone section and updated corresponding references. Also updated structure of ZoneEntry.

Exported MX list from STS policy retaining original * for wildcard names (as the opposite to match list which uses .domain.tld format for wildcards).

CC @wietse-postfix for review.

@Snawoot Snawoot self-assigned this Feb 4, 2025
@Snawoot
Copy link
Owner Author

Snawoot commented Feb 4, 2025

For gmail.com domain I get following response:

user@ws:~/src/postfix-mta-sts-resolver> /usr/sbin/postmap -q gmail.com socketmap:inet:127.0.0.1:8461:myzone
secure match=.gmail-smtp-in.l.google.com:gmail-smtp-in.l.google.com servername=hostname policy_type=sts policy_domain=gmail.com mx_host_pattern=gmail-smtp-in.l.google.com mx_host_pattern=*.gmail-smtp-in.l.google.com

Copy link

@wietse-postfix wietse-postfix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have one trivial suggestion, and one that may be harder.

Copy link

@wietse-postfix wietse-postfix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found the approve button.

@Snawoot Snawoot merged commit afaaa01 into master Feb 4, 2025
2 checks passed
@Snawoot
Copy link
Owner Author

Snawoot commented Feb 4, 2025

@wietse-postfix Thank you for the collaboration!

@Snawoot Snawoot deleted the tlsrpt_zone_prop branch February 4, 2025 22:38
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

Successfully merging this pull request may close these issues.

None yet

2 participants