Page MenuHomeFreeBSD

Add a section to blacklistd to the PF chapter of the handbook
Needs ReviewPublic

Authored by bcr on Jun 9 2019, 12:44 PM.

Details

Summary

This patch adds a description on how to use blacklistd together with PF to the handbook.
The rendered output can be found starting at section 30.3.2.5.

I'm not really sure about the proper placement of this. It could go at the end of the chapter, it's own chapter or where it is now. Where it is currently, having too many sub-sub-subsections does not look good to me.

Test Plan
  1. Apply the patch in the local doc directory
  2. Build the handbook afterwards
  3. Check the output of firewalls-pf.html

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

bcr created this revision.Jun 9 2019, 12:44 PM
kristof added inline comments.Jun 15 2019, 2:38 PM
en_US.ISO8859-1/books/handbook/firewalls/chapter.xml
1052

"a network that should not be accessed"
I'm not sure what you mean here.

1096

Rules either belong to...

1097

which applies to?

1136

SSH primarily uses TCP?

(And not just primarily. I don't think you can do SSH over UDP.)

bcr updated this revision to Diff 58702.Jun 16 2019, 2:26 PM

Update diff to address Kristof's comments.

Given that it's possible to use blacklistd with ipfw as well (at least, I believe it is), it should probably get its own chapter, with pf and ipfw subchapters.

bcr updated this revision to Diff 61691.Thu, Sep 5, 2:26 PM

Sorry to keep you waiting for an update for so long.
I've refactored the blacklistd section into it's own chapter at the end of the firewalls chapter. There are no links from the PF chapter (or any other) to it yet.

Other than the typo this looks good to me.

en_US.ISO8859-1/books/handbook/firewalls/chapter.xml
4108

Expire?

bcr updated this revision to Diff 61906.Tue, Sep 10, 7:12 PM

Good catch with the "expore", which should indeed be "expire". For some reason, igor did not catch that.
I think we can risk committing it to the handbook now and fix other things in a followup commit.