Skip to content

fix: detect Mimecast South Africa, USB and Offshore MX regions - #32

Merged
Zacgoose merged 1 commit into
mainfrom
dev
Aug 24, 2026
Merged

fix: detect Mimecast South Africa, USB and Offshore MX regions#32
Zacgoose merged 1 commit into
mainfrom
dev

Conversation

@Zacgoose

Copy link
Copy Markdown
Contributor

The Mimecast MxMatch only recognised the two-letter *.mimecast.com hostnames, so three documented regions were never identified by the domain analyser:

  • South Africa uses za-smtp-inbound-N.mimecast.co.za (different TLD).
  • The US (B) region uses a three-letter prefix (usb-), which the [a-z]{2} class mis-captured as 'sb'.
  • Offshore (Jersey) uses *.mimecast-offshore.com.

Broaden the prefix to 2-3 chars, anchor the match, and accept the .co.za TLD; the za prefix still resolves the SPF include to za._netblocks.mimecast.com, which is the include those domains publish. Add a separate Mimecast Offshore provider for *.mimecast-offshore.com with an empty SpfInclude, because that region's netblock host is not publicly documented and every candidate resolves NXDOMAIN - leaving it empty skips SPF-include validation rather than asserting a wrong expected include.

The Mimecast MxMatch only recognised the two-letter *.mimecast.com
hostnames, so three documented regions were never identified by the
domain analyser:

- South Africa uses za-smtp-inbound-N.mimecast.co.za (different TLD).
- The US (B) region uses a three-letter prefix (usb-), which the
  [a-z]{2} class mis-captured as 'sb'.
- Offshore (Jersey) uses *.mimecast-offshore.com.

Broaden the prefix to 2-3 chars, anchor the match, and accept the
.co.za TLD; the za prefix still resolves the SPF include to
za._netblocks.mimecast.com, which is the include those domains
publish. Add a separate Mimecast Offshore provider for
*.mimecast-offshore.com with an empty SpfInclude, because that
region's netblock host is not publicly documented and every
candidate resolves NXDOMAIN - leaving it empty skips SPF-include
validation rather than asserting a wrong expected include.
@Zacgoose
Zacgoose requested a review from a team August 24, 2026 16:45
@Zacgoose
Zacgoose merged commit b7c1235 into main Aug 24, 2026
6 checks passed
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.

2 participants