Deprecate bitmask DNSBL records.

We don't use these in any of our docs or example configs and almost
everyone uses the record type. Using a bitmask is also a source of
footguns because users don't really understand how bit masks
actually work.
This commit is contained in:
Sadie Powell 2025-02-27 22:47:27 +00:00
parent 72ff58f7ed
commit 98397935cb

View File

@ -139,13 +139,16 @@ public:
{ "zline", Action::ZLINE },
});
const std::string typestr = tag->getString("type");
const std::string typestr = tag->getString("type", "record", 1);
if (insp::equalsci(typestr, "bitmask"))
{
type = Type::BITMASK;
bitmask = tag->getNum<unsigned int>("bitmask", 0);
records = 0;
ServerInstance->Logs.Normal(MODNAME, "<dnsbl> tag at {} specifies a bitmask. This is deprecated and will be removed in the next major version of InspIRCd, consider migrating to records instead.",
tag->source.str());
}
else if (insp::equalsci(typestr, "record"))
{