status 2023q3: add pf status report

[naman_freebsdfoundation.org]

Looks good to me! Thanks for the report.

[emaste]

website/content/en/status/report-2023-04-2023-06/pf.adoc

28

Should also include Sponsored by: The FreeBSD Foundation for Naman's work.

[emaste]

website/content/en/status/report-2023-04-2023-06/pf.adoc

3

Maybe add a brief introduction, like

pf is one of the firewalls included in FreeBSD, and is probably the most popular. pf was created by the OpenBSD project and subsequently ported to FreeBSD.

[kp]

• add SCTP work
• review remarks

[emaste]

website/content/en/status/report-2023-04-2023-06/pf.adoc

45

These should not have the [Dxxxxx] I believe, so that the URL itself is displayed
Right now I think it would render as:

D40911 URL: D40911

while we want

D40911 URL: https://reviews.freebsd.org/D40911

[emaste]

LGTM aside from the link nit

[kp]

fix links

[salvadore]

The report needs some work on formatting:

• please use one sentence per line;
• please respect the usual sections sorting (see other reports to find many examples): title, links, contacts, body of the reports, sponsors, if any.

Also please consider adding some references to manual pages where appropriate using the man macro, e.g. man pf[4].

website/content/en/status/report-2023-04-2023-06/pf.adoc

45

Please add a pair of empty brackets [] at the end of each second URL, e.g.

link:https://reviews.freebsd.org/D40911[D40911] URL link:https://reviews.freebsd.org/D40911[] +

[kp]

I'm sorry, I don't have the cycles to learn another markup language.

[debdrup]

Spotted some minor nits, but these can be fixed when it's pushed.

website/content/en/status/report-2023-04-2023-06/pf.adoc

31–32

AsciiDoc needs a plus at the end of each sentence that's supposed to form part of the same paragraph, but be listed on one line (unless it's in bullet point form).

41

Also needs a plus here.

47–55

I don't know if it matters, but I think it'd be a good idea to have a space before the plus charracter.

Last line doesn't need the plus character.

[emaste]

Update markup

[emaste]

reorder sections

[salvadore]

Thanks, a few more changes and I think it is ready.

website/content/en/status/report-2023-04-2023-06/pf.adoc

1

Maybe use a capital P: "Pf Improvements"?

20

I suggest using man:pf[4] in the first pf occurrence.

23

s/backport/Backport/

Since "Improvements" has been written with a capital "I", probably it woule be better to also write "Syntax" with a capital "S" for consistency.

26

s/configration/configuration/

28

==== pfsync Protocol Versioning

or

==== Pfsync Protocol Versioning

30

man:pfsync[4]

33

==== pfsync: transport over IPv6

or

==== Pfsync: Transport Over IPv6

[emaste]

suggested updates

[salvadore]

Approved, just please remember to fix the typo at commit time.

Thanks @kp and @emaste for the report.

website/content/en/status/report-2023-04-2023-06/pf.adoc

26

Sorry @emaste, you missed this comment.