It is up to every single node operator to decide what transactions to forward and having more tools to decide on this is a good thing to have.

The only objectional feature to me is that there is a small default set of blocked addresses and that it defaults to "enabled" with no further notice. This is unexpected, as this is not a feature of vanilla bitcoin-core, so if you ship it, it should either give a clear indication what has been added or be disabled by default, to match upstream more closely.