Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi Folks,
As mentioned in email, I have been working on a HashedRPZ patch for unbound.
With the recent RPZ changes and other work, this took a bit longer than I wanted, but hereby it is: HashedRPZ support for unbound.
This will very likely need various improvements, amongst others:
thus comments welcome for these and likely other points.
and in the future also the inline key support and possibly some speed improvements (dname_str and sldns_str2wire_dname_buf usage are likely not superb for speed reasons; but considering HashedRPZ works on a full human-version label and not wire-label, one of those is unavoidable from my attempts at avoiding the conversion).
One way to test this, if there is a zone named rpz.example.net, use the hasher command to generate a hash:
which should result in:
Then in a zonefile, enter:
and, if that zone file is then loaded with the following snippet in unbound.conf:
and one performs a
dig @ns1.example.net. blocked.example.com
should return NXDOMAIN(Having an actual blocked.example.com with a TXT record saying "you should not see me" or similar can be beneficial for testing)
Regards,
Jeroen