Skip to content

Add Mailname email template (mailname.top / email) #1371

Open
huangfeihong-cloud wants to merge 3 commits into
Domain-Connect:masterfrom
huangfeihong-cloud:master
Open

Add Mailname email template (mailname.top / email) #1371
huangfeihong-cloud wants to merge 3 commits into
Domain-Connect:masterfrom
huangfeihong-cloud:master

Conversation

@huangfeihong-cloud

@huangfeihong-cloud huangfeihong-cloud commented Jul 11, 2026

Copy link
Copy Markdown

Add Mailname email template

Description

Type of change

Please mark options that are relevant.

  • New template
  • Bug fix (non-breaking change which fixes an issue in the template)
  • New feature (non-breaking change which adds functionality to the template)
  • Breaking change (fix or feature that would cause existing template behavior to be not backward compatible)

How Has This Been Tested?

Please mark the following checks done

  • Template functionality checked using Online Editor
  • Template file name follows the pattern <providerId>.<serviceId>.json
  • resource URL provided with logoUrl is actually served by a webserver

Checklist of common problems

Mark all the checkboxes after conducting the check. Comment on any point which is not fulfilled.
See Template Quality Guidelines for details and rationale on each rule.

  • syncPubKeyDomain is set — this is mandatory; omitting it requires explicit justification in the PR description or the PR will be rejected
  • warnPhishing is not set alongside syncPubKeyDomain — the two must not appear together
  • syncRedirectDomain is set whenever the template uses redirect_uri in the synchronous flow
  • no TXT record contains SPF content ("v=spf1 ...") — use the SPFM record type instead
  • txtConflictMatchingMode is set on every TXT record that must be unique per label or content prefix (e.g. DMARC)
  • no variable is used as a bare full record value (e.g. @ TXT "%foo%") unless necessary — prefer @ TXT "service-foo=%foo%"; if bare, justify in the PR description
  • no bare variable is used as the full host label — the non-variable parts are fixed to limit misuse (e.g. %dkimkey%._domainkey, not %dkimhost%); if bare, justify in the PR description
  • no variable is used in the host field to create a subdomain — use the host parameter or multiInstance instead
  • %host% does not appear explicitly in any host attribute
  • essential is set to OnApply on records the end user may need to modify or remove without breaking the template (e.g. DMARC)

Online Editor test results

Editor test link(s):
[Test apply mail.example.com]https://domainconnect.paulonet.eu/dc/free/templateedit?token=H4sIABIuU2oC%2F%2B1Wa2%2FiOBT9K5GlSrvaQM2jFKiQGqBlGDYdXp1ht6qQSQx4SeLUdihp1f3te80zPEaa%2BbQdqV9I5HvPfZzce%2FArUtQPPaIoKr%2BiUPA5c6louqiMfMK8gPg0rXiIzK3tDo7Aaq%2BtYJFUzJlDlyCqUbuzA2fjZm2eUyEZD1A5YyKPT%2Fi98MBtqlQoy%2Bfnyczn2pyW8wmgXCodwUK1RKIeVUYUGtvYy9TGb%2FbANHrtW9Oot5o2%2FNpWt%2Fa7wQMj5pEwXA5egS4wDpx2NGrRuL46OmpYe3SpywR11NaHuD4L0geeUy5Vlz5F4AocKBFREwGKC1ei8sMrUnG4ZGGw9oX3a80oZ4GSfb5OnT4inHHBVAwkYRMpBQzlChi%2FmduA0Ka9H1KG427kUalDLgyCTuP6g%2F4OdubOmC%2BpB11ycZYerhia0VgTThQBl3lFc5m5MmYVIcmVEVaWIPA5S2aA14Wq8WDsMUfZRDlTFkxs7uqUlueh71UwdH0inL10%2BqNldKKniAgSKBbQK0NEpKI5UrwcAtInUlFxfbYq%2BKcqMRGVkkJYosfuS2CFoRd%2Ft77rXWmbT5SCAWbjuHK2eio%2Boz9eQVvQMVugky5r23Em9Pb4ZqIXHtDhbrQeobDNZNIFgUWmaYf7u9LX6zYRPAqHbIMJgVNf6oXfoB%2F24I8b%2FMMqgE6TGBJ9LjObUz0pcGA3G2Pbwo1a76nRa45y9c5N1ercW1a%2BcWfVa1XWaVUnnbqiUgEik83lLwqXxRImI8el48mU%2FTPz%2FICHT0KqaP68iF%2Bsaq1%2Bc9v41Pzc%2BtO%2B%2B9LudHv9%2B6%2FfBn%2F9jbfwZt3qWFVdSuJD6HIgLDhBZPBDmjk2CbigQwlPoiJBN3vqR55iQ%2FJMdkeuMyR6IIBoCVYId7DDwUrV1uQmRuNwhRPjsL%2FNQ9fR9DMtmc5oPKKFsZNycm4plc9gnBqVyDiVd7I4716WRtkRTejvKW0%2BpcD7E5Ccd8t7JrFEb0fzfrKveQVEJWNoOTH%2BJcvlSbT1rhuRmYScpQ%2FbOtS0%2F3WCfx1SV2p9zOZPSPa%2BVL2Xxrd%2FAz%2B2F4cCvac47%2FBzPpog%2FR9C9iFkH0L2IWS%2FtJDBbU%2BSOXWHRLtmcbaQwpepTLaPC2WcLV8U00VcyOfwHxiXMdYtwKSvmn2Fm2DyDoiaxRee%2FfRNNF%2Fwndf63JsPbhZF78LynH6uH93G7RLJhbX212nzHq7g%2FwHPBlXLLg8AAA%3D%3D

Add Mailname email template
@github-actions github-actions Bot added no syncPubKeyDomain syncPubKeyDomain not set. Template is not secured. Checklist of common problems not complete See PR template and mark *all* checkboxes, even if not applicable. Explain any discrepancies. warnPhishing warnPhishing is set. Template is not secured. PR description incomplete The PR description template was not filled in at all, altered or filled in improperly. Test links missing No test links from Online Editor provided labels Jul 11, 2026
@github-actions

Copy link
Copy Markdown

JSON Filename Check Passed

@github-actions

Copy link
Copy Markdown

JSON Schema Validation Passed

@github-actions

github-actions Bot commented Jul 11, 2026

Copy link
Copy Markdown

Linter OK

mailname.top.email.json

Level Code Note
info DCTL1037 hostRequired template should be combined with NS or CNAME record that uses host @ or empty

@github-actions github-actions Bot added automerge-possible Label indicating, that restrictive linter checks all pass. For now it's PoC. and removed no syncPubKeyDomain syncPubKeyDomain not set. Template is not secured. warnPhishing warnPhishing is set. Template is not secured. labels Jul 11, 2026
github-actions[bot]
github-actions Bot previously approved these changes Jul 11, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Linter passed all checks — approving for auto-merge.

@github-actions github-actions Bot removed the Checklist of common problems not complete See PR template and mark *all* checkboxes, even if not applicable. Explain any discrepancies. label Jul 11, 2026
Update mailname.top.email.json
@github-actions github-actions Bot added hostRequired hostRequired is set and removed automerge-possible Label indicating, that restrictive linter checks all pass. For now it's PoC. PR description incomplete The PR description template was not filled in at all, altered or filled in improperly. Test links missing No test links from Online Editor provided labels Jul 12, 2026
@github-actions

Copy link
Copy Markdown

PR Description Check Passed

All required sections are filled in correctly.

Details
  OK  Type of change: 1/4 checkboxes ticked
  OK  How Has This Been Tested?: all 3 checkboxes ticked
  OK  Checklist of common problems: 10/10 checkboxes ticked
  OK  Online Editor test results: 1 link(s) found
  OK  Template coverage: all 1 template(s) covered
Labels to remove: Checklist of common problems not complete, Forged editor links, PR description incomplete, Test links missing

PR description check PASSED

@pawel-kow pawel-kow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

pls give this PR a meaningful name.
Otherwise looks ok.

@github-actions github-actions Bot added the review issues Changes requested from PR review label Jul 12, 2026
@huangfeihong-cloud huangfeihong-cloud changed the title Add files via upload Add Mailname email template (mailname.top / email) Jul 12, 2026
@huangfeihong-cloud

Copy link
Copy Markdown
Author

pls give this PR a meaningful name. Otherwise looks ok.

Thanks for the review! I've renamed the PR to a meaningful title. Please let me know if anything else is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hostRequired hostRequired is set review issues Changes requested from PR review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants