Deprecate and remove support for SPF record type
Description
The SPF record type has been deprecated since years. In July, we announced we were going to discontinue support for the SPF record type.
Starting from Sept 8th, we will no longer host SPF records.
We should align erldns and drop support for SPF. This is going to be a major version change.
Expected Behavior or Outcome
- Support for the SPF record type is removed from erldns.
- dns_erlang should not be changed
Acceptance Criteria
- New erldns major version is released, dropping support for SPF record type.
This PR removes any SPF records references in the dnstest library: https://github.com/dnsimple/dnstest/pull/34. The change is very simple. @weppos @DXTimer please have a look so we can merge it and proceed.
Second PR ready for review: https://github.com/dnsimple/erldns/pull/282. Quite small, since the only type-specific code was related to encoding/decoding. @weppos @DXTimer
The change is now merged into main branch. @weppos do you want me to create v9.0.0.RC1 release? Also, I see that on GitHub we're missing release v8.1.0 even though the tag was created before - https://github.com/dnsimple/erldns/releases. I could create this release as well beforehand.
The change is now merged into
mainbranch. @weppos do you want me to createv9.0.0.RC1release? Also, I see that on GitHub we're missing releasev8.1.0even though the tag was created before - https://github.com/dnsimple/erldns/releases. I could create this release as well beforehand.
It looks like we use v9.0.0-rc1, see https://github.com/dnsimple/erldns/tags
I don't believe we create a GitHub "release" for each release tag, just for the major. We can skip 8.1.0.
Release v9.0.0-rc1 is now available at hex.pm. After it is tested we can release a stable version and then we'll close this issue.
Unassigning myself since I will no longer work on this. The only thing left to meet the acceptance criteria is to release a stable version (currently it is a release candidate).
@NelsonVides can you please take a look at this ticket. I was under the impression we removed the SPF completely. Perhaps we just needed to rollout v9 (which was done), and if so then this ticket should be closed.
It seems perfectly well removed to me and indeed, releasing v9 was enough (v9-rc1 to be precise).