github-personal/Mailu
1
0
Fork 0
mirror of https://github.com/outbackdingo/Mailu.git synced 2026-01-27 18:19:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge #3150

Browse Source 
3150: Add trailing semicolon for DMARC authorisation record r=mergify[bot] a=su-ex

## What type of PR?

bug-fix

## What does this PR do?

Add trailing semicolon for DMARC authorisation records, which seems to be necessary: https://stackoverflow.com/a/72463456
I ran into this when testing my domain with internet.nl.


Co-authored-by: su-ex <codeworks@supercable.onl>
Co-authored-by: Florent Daigniere <nextgens@freenetproject.org>
This commit is contained in:
bors-mailu[bot]
2024-02-05 17:36:59 +00:00
committed by GitHub
parent e0c5dc5fe5 98a6bab0ca
commit 9d0325fa15
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
core/admin/mailu/models.py
View File

@@ -250,7 +250,7 @@ class Domain(Base):
""" return DMARC report record for mailu server """
if self.dkim_key:
domain = app.config['DOMAIN']
return f'{self.name}._report._dmarc.{domain}. 600 IN TXT "v=DMARC1"'
return f'{self.name}._report._dmarc.{domain}. 600 IN TXT "v=DMARC1;"'
@cached_property
def dns_autoconfig(self):

1
towncrier/newsfragments/3150.bugfix Normal file
View File

@@ -0,0 +1 @@
Add a semicolum to DMARC records